From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | 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 14:52:54 |
Message-ID: | 6542.1267023174@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Magnus Hagander <magnus(at)hagander(dot)net> writes:
> 2010/2/24 Tom Lane <tgl(at)postgresql(dot)org>:
>> Log Message:
>> -----------
>> Un-break pg_dump for the case of zero-column tables.
>>
>> This was evidently broken by the CREATE TABLE OF TYPE patch. It would have
>> been noticed if anyone had bothered to try dumping and restoring the
>> regression database ...
> Is there a point in doing that at the end of "make check"? Or as a
> separate step on the buildfarm?
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.
Still, if anyone who knows the buildfarm code cares to try that,
I'd be all for it.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | User Gsmet | 2010-02-24 15:01:03 | pgfouine - pgfouine: fixed [#1010689] undefined method |
Previous Message | Robert Haas | 2010-02-24 14:51:38 | Re: pgsql: Remove pre-7.4 documentaiton mentions, now that 8.0 is the oldest |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2010-02-24 15:01:47 | Re: A thought on Index Organized Tables |
Previous Message | Robert Haas | 2010-02-24 14:51:38 | Re: pgsql: Remove pre-7.4 documentaiton mentions, now that 8.0 is the oldest |