Re: [PATCHES] -HEAD pg_dumpall broken against older backends
Stefan Kaltenbrunner wrote: looks like somebody forgot to test some changes to the pg_dumpall code in Revision 1.70 against 8.1 installations - resulting in the following syntax error: Dump output is never expected to be backward compatible. -- Peter Eisentraut http://developer.postgresql.org/~petere/ ---(end of broadcast)--- TIP 5: don't forget to increase your free space map settings
Re: [PATCHES] -HEAD pg_dumpall broken against older backends
Peter Eisentraut [EMAIL PROTECTED] writes: Stefan Kaltenbrunner wrote: looks like somebody forgot to test some changes to the pg_dumpall code in Revision 1.70 against 8.1 installations - resulting in the following syntax error: Dump output is never expected to be backward compatible. But pg_dumpall should be able to extract a dump from an older server. regards, tom lane ---(end of broadcast)--- TIP 4: Have you searched our list archives? http://archives.postgresql.org
Re: [PATCHES] -HEAD pg_dumpall broken against older backends
Peter Eisentraut wrote: Stefan Kaltenbrunner wrote: looks like somebody forgot to test some changes to the pg_dumpall code in Revision 1.70 against 8.1 installations - resulting in the following syntax error: Dump output is never expected to be backward compatible. We don't expect pg_dumpall to be issuing invalid SQL at any time, though. That's what Stefan has uncovered. The code in question is only called when it's run against an older server. cheers andrew ---(end of broadcast)--- TIP 2: Don't 'kill -9' the postmaster
Re: [PATCHES] -HEAD pg_dumpall broken against older backends
Peter Eisentraut wrote: Stefan Kaltenbrunner wrote: looks like somebody forgot to test some changes to the pg_dumpall code in Revision 1.70 against 8.1 installations - resulting in the following syntax error: Dump output is never expected to be backward compatible. yeah - but if you take a look at the code/patch you will see that the problem in this case is that pg_dumpall generates invalid SQL if it is operating against an older backend which is an obvious typo/thinko in this case. It has nothing to do with generating backwards compatible dumps - and using the pg_dumpall from the (newer) target version for upgrades has been recommended for a while (or rather is documented to work in the manual) I think. Stefan ---(end of broadcast)--- TIP 5: don't forget to increase your free space map settings
Re: [PATCHES] -HEAD pg_dumpall broken against older backends
Stefan Kaltenbrunner wrote: Hi! looks like somebody forgot to test some changes to the pg_dumpall code in Revision 1.70 against 8.1 installations - resulting in the following syntax error: patch applied, thanks andrew ---(end of broadcast)--- TIP 2: Don't 'kill -9' the postmaster