Skip site navigation (1) Skip section navigation (2)

Re: [COMMITTERS] pgsql: Un-break pg_dump for the case of zero-column tables.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Un-break pg_dump for the case of zero-column tables.
Date: 2010-02-24 16:30:50
Message-ID: 8596.1267029050@sss.pgh.pa.us (view raw or flat)
Thread:
Lists: pgsql-committerspgsql-hackers
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> Tom Lane wrote:
>> I think it would make sense to add it as a buildfarm phase, probably
>> after installcheck not check so you still have a working postmaster.
>> I'm not sure how easy it'd be to automate though.  What I usually do
>> is make a text dump, restore the dump into an empty DB (watching for
>> errors), dump that, and diff the two dumps.  However the expected
>> diff is not empty because of some tests that intentionally stress
>> inheritance column order, and I'm not sure whether it is stable
>> enough to use a simple expected-result comparison.

> One for this purpose would have to be somewhat more involved than this 
> example, since we'd need to invent a success measure, which "make" 
> already gives us for free, and also there are more steps involved than 
> running a single "make" command. But it should not be terribly 
> difficult  for anyone who wants to try.

Actually, given the need for an expected-result file, which could be
expected to require maintenance, I suspect that most of the
infrastructure for this would have to go into the core source tree
anyway.  Perhaps a "make install-check-pg_dump" target in src/regress?

			regards, tom lane

In response to

pgsql-hackers by date

Next:From: Jaime CasanovaDate: 2010-02-24 16:37:34
Subject: Re: Determining client_encoding from client locale
Previous:From: Kevin GrittnerDate: 2010-02-24 16:28:08
Subject: Re: A thought on Index Organized Tables

pgsql-committers by date

Next:From: Tom LaneDate: 2010-02-24 16:59:00
Subject: Re: pgsql: Revert removal of pre-7.4 documenation behavior mentions.
Previous:From: Bruce MomjianDate: 2010-02-24 16:27:40
Subject: pgsql: Document that pg_restore -t can use -n for its schema

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group