<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul 1, 2013 at 3:24 PM, David Ripton <span dir="ltr"><<a href="mailto:dripton@redhat.com" target="_blank">dripton@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 07/01/2013 12:49 PM, Brant Knudson wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
'Stackers -<br>
<br>
I've got a review up in Keystone that converts tables from MyISAM to<br>
InnoDB [0], which I patterned after a change in Nova. One of the<br>
comments in the review is suggesting that the migrate_version table<br>
should also be changed. The reason I didn't include migrate_version is<br>
because that's the way Nova did it, but other than that I don't know why<br>
migrate_version should not be converted. The Nova code is pretty<br>
explicit that migrate_version isn't changed [1]. Maybe somebody who<br>
knows MySQL or SQLAlchemy-migrate better than I do can come up with a<br>
reason why migrate_version shouldn't be changed from MyISAM to InnoDB.<br>
<br>
[0] <a href="https://review.openstack.org/#/c/33102/" target="_blank">https://review.openstack.org/#<u></u>/c/33102/</a> - Use InnoDB for MySQL<br>
[1]<br>
<a href="https://github.com/openstack/nova/blob/master/nova/tests/db/test_migrations.py#L331" target="_blank">https://github.com/openstack/<u></u>nova/blob/master/nova/tests/<u></u>db/test_migrations.py#L331</a><br>
</blockquote>
<br></div></div>
sqlalchemy-migrate relies on the migrate_versions table, so modifying it from within a sqlalchemy-migrate script is scary.  And it's a tiny table that's only used during DB migrations, so I doubt you'd see any actual benefit.</blockquote>
<div><br></div><div style>+1; we don't explicitly create the migrate_versions table in migrations, so we shouldn't be managing it either. If there's an issue in that table, I'd say it's most likely on sqlalchemy-migrate or the deployer to fix, not keystone's migrations.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
David Ripton   Red Hat   <a href="mailto:dripton@redhat.com" target="_blank">dripton@redhat.com</a><br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><br></div>-Dolph
</div></div>