Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update database and tables #2

Merged
merged 1 commit into from Feb 7, 2014

Conversation

sublimino
Copy link
Member

Update database and tables to InnoDB, utf8, auto_inc 1 and unsigned on some ints

@graphiclunarkid
Copy link
Member

From the IRC meeting in #orgtechvols this evening:

(19:33:48) graphiclunarkid: One thing I wanted to ask, actually, is how we get more eyes on the middleware code. I see there's an outstanding pull request from someone from a couple of weeks ago.
(19:34:06) graphiclunarkid: I would just accept it - but I don't want to do so without someone reviewing the code first.
(19:34:15) graphiclunarkid: It looks pretty non-controversial though.
(19:34:40) dantheta1: Yep - that's the utf and innodb changes in the MySQL table structure. I'd say they're good to go.
(19:34:57) graphiclunarkid: #2
(19:35:11) graphiclunarkid: Oh cool, OK then. I'll merge them.

graphiclunarkid added a commit that referenced this pull request Feb 7, 2014
Update database and tables
@graphiclunarkid graphiclunarkid merged commit fdd46ab into openrightsgroup:master Feb 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants