Re: Bug in CVS pg_dump against 7.0.x - Mailing list pgsql-patches

From Tom Lane
Subject Re: Bug in CVS pg_dump against 7.0.x
Date
Msg-id 11336.1084937774@sss.pgh.pa.us
Whole thread Raw
In response to Re: Bug in CVS pg_dump against 7.0.x  (Christopher Kings-Lynne <chriskl@familyhealth.com.au>)
Responses Re: Bug in CVS pg_dump against 7.0.x  (Christopher Kings-Lynne <chriskl@familyhealth.com.au>)
List pgsql-patches
Christopher Kings-Lynne <chriskl@familyhealth.com.au> writes:
>> Do we know that is always true?  What is the issue that 7.0 needs this
>> and newer released don't, and how are we sure this will not break some
>> strange cases in post-7.0 releases?

> It's not a problem, since that code path is only ever executed when
> dumping a 7.0 backend.  It's in the MyFormatType function in pg_dump.c
> that is used whenever the backend doesn't have its own format_type function.

Right.  The patch looked dangerous to me too, until I understood the
context.  As-is, I think it's fine -- indeed, I think we ought to
backpatch it into 7.4 branch too.

            regards, tom lane

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: More network functions...
Next
From: Christopher Kings-Lynne
Date:
Subject: Re: Bug in CVS pg_dump against 7.0.x