Skip to content

Commit

Permalink
srm: Fix renaming of count column
Browse files Browse the repository at this point in the history
Motivation:

The attempt to rename the oddly capitalized count column of the lsrequests
table didn't work as liquibase itself escaped our quotes.

Modification:

Since the other columns are already lower case in postgresql and since
our HSQLDB databases are always recreated for testing, this patch drops
the changeset entirely.

Result:

SRM schema migration of existing schema works.

Target: trunk
Require-notes: no
Require-book: no
Acked-by: Paul Millar <paul.millar@desy.de>
Patch: https://rb.dcache.org/r/8702/
  • Loading branch information
gbehrmann committed Oct 30, 2015
1 parent 3196ffe commit cd565a6
Showing 1 changed file with 1 addition and 6 deletions.
Expand Up @@ -478,7 +478,7 @@
<column name="explanation" type="varchar(32762)"/>
<column name="longformat" type="integer"/>
<column name="numoflevels" type="integer"/>
<column name='"count"' type="integer"/>
<column name="count" type="integer"/>
<column name="lsoffset" type="integer"/>
</createTable>

Expand Down Expand Up @@ -1011,11 +1011,6 @@
<rollback/>
</changeSet>

<changeSet id="9" author="behrmann">
<comment>Rename count column to use the same capitalization as for other columns</comment>
<renameColumn tableName="lsrequests" oldColumnName='"count"' newColumnName="count"/>
</changeSet>

<changeSet id="10" author="behrmann">
<comment>Drop indexes on unused nextjob columns</comment>

Expand Down

0 comments on commit cd565a6

Please sign in to comment.