Hacker News new | ask | show | jobs
by jzawodn 5385 days ago
Hmm. I see "ENGINE=MyISAM" in the code. Nowadays I can't stomach using MyISAM for anything I care about not having some sort of corruption. Given sufficient time and scale, I've seen far too many issues with MyISAM.

It'd be great to be able to specify that in the code so the user can opt for InnoDB (or engine of their choice). Perhaps a pull request is in my future...
2 comments
cosimo 5384 days ago
I seem to remember that the MySQL behavior this is based on will only work with the MyISAM engine. In fact, Flickr is using MyISAM for this, IIRC. Anyway, your changes make perfect sense, so I merged them.

Will try what happens to the test suite with InnoDB.

link
cosimo 5384 days ago
Actually it works just fine ;)
link
ginzasparrow 5385 days ago
You could've cloned the repo and started hacking on it in the time it took you to write that.
link
jzawodn 5385 days ago
Agreed.

https://github.com/cosimo/TicketServer/pulls

link
ginzasparrow 5385 days ago
Damn you for undermining my snarky comment. Cheers.
link
cosimo 5384 days ago
Merged too.
link