Ticket #1606 (closed defect: fixed)

Opened 15 months ago

Last modified 15 months ago

upgrade script failure: error can not change name of input parameter "new_srid"

Reported by: robe Owned by: strk
Priority: blocker Milestone: PostGIS 2.0.0
Component: build/upgrade/install Version: trunk
Keywords: Cc:

Description

Get this error when trying to upgrade from alpha5 to alpha6 using

ALTER EXTENSION postgis UPDATE TO "2.0.0alpha6";

Haven't checked if similar issue with regular upgrade.

Attachments

restore_par_name_UpdateGeometrySRID.patch Download (3.4 KB) - added by mwtoews 15 months ago.
fix-up from #1596

Change History

Changed 15 months ago by robe

  • owner changed from pramsey to strk
  • component changed from postgis to build/upgrade/install
  • summary changed from error can not change name of input parameter "new_srid" to upgrade script failure: error can not change name of input parameter "new_srid"

Same issue with 9.0 build

I think its this function -- we need to drop it first as part of upgrade

CREATE OR REPLACE FUNCTION UpdateGeometrySRID(catalogn_name varchar,schema_name varchar,table_name varchar,column_name varchar,new_srid_in integer)

Changed 15 months ago by mwtoews

This error stemmed from #1596. The last parameter in UpdateGeometrySRID, new_srid, was renamed to new_srid_in. The rational was to somewhat mimic AddGeometryColumn. This can be modified back, if required. (Another internal variable name like new_srid_mod would need to be used within UpdateGeometrySRID, since the IN parameters are CONST and this var may change to unknown_srid).

Changed 15 months ago by mwtoews

fix-up from #1596

Changed 15 months ago by mwtoews

If it is preferred to keep the old parameter names, use the attached patch.

Changed 15 months ago by strk

This is the kind of regression our testsuite would hardly ever get

Changed 15 months ago by strk

Changing again would mean that the upgrade will fail for anyone going from alpha6 to alpha7, but those going from alpha5 to alpha7 would be fine.

Better go with the drop, probably.

Changed 15 months ago by strk

  • status changed from new to closed
  • resolution set to fixed

Fixed with r9275

Note: See TracTickets for help on using tickets.