]> granicus.if.org Git - postgresql/commit
Fix contrib/citext's upgrade script to handle array and domain cases.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 11 May 2012 19:22:37 +0000 (15:22 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 11 May 2012 19:22:37 +0000 (15:22 -0400)
commit2c1382d96c575a79a2ce014dda90fd1de78b3612
treef8458bc9f54f655bdcf4acff450775c2755b17bb
parent2ca6bac892aa650154f591c730359a968bea726a
Fix contrib/citext's upgrade script to handle array and domain cases.

We previously recognized that citext wouldn't get marked as collatable
during pg_upgrade from a pre-9.1 installation, and hacked its
create-from-unpackaged script to manually perform the necessary catalog
adjustments.  However, we overlooked the fact that domains over citext,
as well as the citext[] array type, need the same adjustments.  Extend
the script to handle those cases.

Also, the documentation suggested that this was only an issue in pg_upgrade
scenarios, which is quite wrong; loading any dump containing citext from a
pre-9.1 server will also result in the type being wrongly marked.

I approached the documentation problem by changing the 9.1.2 release note
paragraphs about this issue, which is historically inaccurate.  But it
seems better than having the information scattered in multiple places, and
leaving incorrect info in the 9.1.2 notes would be bad anyway.  We'll still
need to mention the issue again in the 9.1.4 notes, but perhaps they can
just reference 9.1.2 for fix instructions.

Per report from Evan Carroll.  Back-patch into 9.1.
contrib/citext/citext--unpackaged--1.0.sql
doc/src/sgml/release-9.1.sgml