pgsql: Fix pg_dumpall to restore its ability to dump from ancient serve

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix pg_dumpall to restore its ability to dump from ancient serve
Date: 2014-11-13 23:20:03
Message-ID: E1Xp3gJ-0003wi-Me@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix pg_dumpall to restore its ability to dump from ancient servers.

Fix breakage induced by commits d8d3d2a4f37f6df5d0118b7f5211978cca22091a
and 463f2625a5fb183b6a8925ccde98bb3889f921d9: pg_dumpall has crashed when
attempting to dump from pre-8.1 servers since then, due to faulty
construction of the query used for dumping roles from older servers.
The query was erroneous as of the earlier commit, but it wasn't exposed
unless you tried to use --binary-upgrade, which you presumably wouldn't
with a pre-8.1 server. However commit 463f2625a made it fail always.

In HEAD, also fix additional breakage induced in the same query by
commit 491c029dbc4206779cf659aa0ff986af7831d2ff, which evidently wasn't
tested against pre-8.1 servers either.

The bug is only latent in 9.1 because 463f2625a hadn't landed yet, but
it seems best to back-patch all branches containing the faulty query.

Gilles Darold

Branch
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/79b2fa5bd2f7d7721651e177c824eb103870d8f5

Modified Files
--------------
src/bin/pg_dump/pg_dumpall.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2014-11-14 00:04:40 pgsql: Adapt valgrind.supp to the XLogInsert() split.
Previous Message Andres Freund 2014-11-13 19:44:01 pgsql: Fix xmin/xmax horizon computation during logical decoding initia