When writing my Dropping ACID: Eating Data in a Web 2.0 Cloud World talk for LCA2011 I came to the realisation that I had forgotten a lot of the things I had worked on in MySQL and MySQL Cluster. So, as a bit of a retrospective as part of the Drizzle7 GA release, I thought I might try and write down a (incomplete) list of the various things I’ve worked on in Drizzle.
I noticed I did a lot of code removal, that’s all fine and dandy but maybe I won’t list all of that… except perhaps my first branch that was merged :)
- First ever branch that was merged: some mysys removal (use POSIX functions instead of wrappers that sometimes have different semantics than their POSIX functions), some removal of NETWARE, build scripts that weren’t helpful (i.e. weren’t what any build team ever used to build a release) and some other dead code removal.
- Improve ‘make test’ time – transactions FTW! (this would be a theme for me over the years, I always want build and test to be faster)
- Started moving functions out into their own plugins, removing the difference between UDFs (User Defined Functions) and builtin functions. One API to rule them all.
- Ahhh compiler warnings (we now build with -Werror) and unchecked return codes from system calls (we now add this to some of our own methods, finding and fixing even more bugs). We did enable a lot of compiler warnings and OMG fix a lot of them.
- Removal of FRM – use a protobuf message to describe the table. The first branch that implemented any of this was merged mid-November. It was pretty minimal, and we still had the FRM around for a little while yet – but it was the beginning of the end for features that couldn’t be implemented due to limitations in the FRM file format. I wrote a few blog entries about this.
- A lot of test fixes for Drizzle
- After relating the story ofÂ â˜ƒ.test (hint: it broke the ability to check out the source tree on Solaris) to Anthony Baxter, he suggested trying something rather nasty… a unicode character above 2^16 – I choseÂ ð„¢ – which at the time didn’t even render on Ubuntu – this was the test case you could only see correctly on MacOS X at the time (or some less broken Linux distro I guess). Some time later, I was actually able to view the test file on Ubuntu correctly.
- I think it was November 1st when I started to work on Drizzle full time – this was rather awesome, although a really difficult decision as I did rather enjoy working with all the NDB guys.
- January sparked the beginning of reading table information from the table protobuf message instead of the FRM file. The code around the FRM file was lovely and convoluted in places – to this day I’m surprised at the low bug count all that effort resulted in. One day I may write up a list of bugs and exploits probably available through the FRM code.
- My hate for C++ fstream started in Feb 2009
- In Feb I finally removed the code to work out (and store) in the FRM file how to display a set of fields for entering data into the table on a VT100 80×24 screen.
- I filed my first GCC bug. The morning started off with a phone call andÂ Brian asking me to look at some strange bug and ended with Intel processor manuals (mmm… the 387 is fun), the C language standard and (legitimately) finding it was a real compiler bug. Two hours and two minutes after filing the bug there was a patch to GCC fixing it – I was impressed.
- By the end of Feb, the FRM was gone.
- I spent a bit of time making Drizzle on linux-sparc work. We started building with compiler options that should be much friendlier to fast execution on SPARC processors (all to do with aligning things to word boundaries). -Wcast-align is an interesting gcc flag
- Moved DDL commands to be in StorageEngine rather than handler for an instance of an open table (now known as Cursor)
- MyISAM and CSV as temporary table only engines – this means we save a bunch of mucking about in the server.
- Amazingly enough, sprintf is dangerous.
- Moved to an API around what tables exist in a database so that the Storage Engines can own their own metadata.
- Move to have Storage Engines deal with the protobuf table message directly.
- Found out you should never assume that your process never calls fork() – if you use libuuid, it may. If it wasn’t for this and if we had param-build, my porting of mtr2 to Drizzle probably would have gone in.
- There was this thing called pack_flag – its removal was especially painful.
- Many improvements to the table protobuf message (FRM replacement) code and format – moving towards having a file format that could realistically be produced by code that wasn’t the Drizzle database kernel.
- Many bug fixes, some acused by us, others exposed by us, others had always been there.
- embedded_innodb storage engine (now HailDB). This spurred many bug and API fixes in the Storage Engine interface. This was an education in all the corner cases of the various interfaces, where the obvious way to do things was almost always not fully correct (but mostly worked).
- Engine and Schema custom KEY=VALUE options.
- Found a bug in the pthread_mutex implementation of the atomics<> template we had. That was fun tracking down.
- started writing more test code for the storage engine interface and upper layer. With our (much improved) storage engine interface, this became relatively easy to implement a storage engine to test specific bits of the upper layer.
- Wrote a CREATE TABLE query that would take over four minutes to run. Fixed the execution time too. This only existed because of a (hidden and undocumented) limitation in the FRM file format to do with ENUM columns.
- Characters versus bytes is an important difference, and one that not all of the code really appreciated (or dealt with cleanly)
- FOREIGN KEY information now stored in the table protobuf message (this was never stored in the FRM).
- SHOW CREATE TABLE now uses a library that reads the table protobuf message (this will later be used in the replication code as well)
- Started the HailDB project
- Updated the innobase plugin to be based on the current InnoDB versions.
- Amazingly, noticed that the READ_COMMITTED isolation level was never tested (even in the simplest way you would ever explain READ_COMMITTED).
- Created a storage engine for testing called storage_engine_api_tester (or SEAPITester). It’s a dummy engine that checks we’re calling things correctly. It exposed even more bugs and strangeness that we weren’t aware of.
- Fixed a lot of cases in the code where we were using a large stack frame in a function (greater than 32kb).
- An initial patch using the internal InnoDB API to store the replication log
- This can be detailed later, it’s still in progress :)
- The big highlight: a release.