On Mon, Feb 07, 2011 at 06:50:14PM +0100, Lukas Fleischer wrote: > On Mon, Feb 07, 2011 at 10:08:54AM -0600, Dan McGee wrote: > > AUR side: > > * Using CHAR as a datatype is absolutely silly in new code, use > > VARCHAR, and why do anything shorter than 255 or 512? > > * On that note, almost all CHAR usages in the current schema are silly > > and should be using VARCHAR- anything on the Packages table, > > PackageCategories, PackageSources, TU_VoteInfo, AccountTypes, > > Username/Email/Passwd/IRCNick on Users. > > I just copied that from another table schema, but you're absolutely > right. I'll work on replacing all that CHAR stuff with VARCHARs later. > Might become a kinda ugly updating process from 1.7.0 to 1.8.0 :)
Fixed. > > * Oh my, I forgot we are still on MyISAM for the AUR. Please to god > > switch to InnoDB and use transactions instead. > > Full ack. I'll talk to Loui about that. Fixed. Everything should be compatible with InnoDB now (talking about what's in my working tree - didn't push to gerolde yet). Also, aurblup uses transactions by default.