pgsql: Fix contrib/citext's upgrade script to handle array and domain c - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix contrib/citext's upgrade script to handle array and domain c
Date
Msg-id E1SSvQO-0002fj-FE@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/63fecc91770f3d494b6fe993bdc198a9faf6fe2d

Modified Files
--------------
contrib/citext/citext--unpackaged--1.0.sql |   97 ++++++++++++++++++++++------
doc/src/sgml/release-9.1.sgml              |   16 +++--
2 files changed, 89 insertions(+), 24 deletions(-)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: Prevent loss of init fork when truncating an unlogged table.
Next
From: Peter Eisentraut
Date:
Subject: pgsql: PL/Python: Adjust the regression tests for Python 3.3