OpenUpgrade Server

Merge lp:~zaber/openupgrade-server/rename-index into lp:openupgrade-server/6.0

  1. rename-index
  2. Merge into 6.0
Proposed by Don Kirkby
Status: Merged
Merged at revision: 3492
Proposed branch: lp:~zaber/openupgrade-server/rename-index
Merge into: lp:openupgrade-server/6.0
Diff against target: 11 lines (+1/-0)
1 file modified
bin/openupgrade/openupgrade.py (+1/-0)
To merge this branch: bzr merge lp:~zaber/openupgrade-server/rename-index
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) Approve
Review via email: mp+114285@code.launchpad.net

Description of the change

Fix bug 1023173 by dropping the index during a column rename.

To post a comment you must log in.
Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/openupgrade/openupgrade.py'
--- bin/openupgrade/openupgrade.py 2012-06-23 14:23:36 +0000
+++ bin/openupgrade/openupgrade.py 2012-07-10 22:03:19 +0000
@@ -105,6 +105,7 @@
105 logger.info("table %s, column %s: renaming to %s",105 logger.info("table %s, column %s: renaming to %s",
106 table, old, new)106 table, old, new)
107 cr.execute('ALTER TABLE "%s" RENAME "%s" TO "%s"' % (table, old, new,))107 cr.execute('ALTER TABLE "%s" RENAME "%s" TO "%s"' % (table, old, new,))
108 cr.execute('DROP INDEX IF EXISTS "%s_%s_index"' % (table, old))
108109
109def rename_tables(cr, table_spec):110def rename_tables(cr, table_spec):
110 """111 """

Subscribers

People subscribed via source and target branches