From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Ian Barwick <ian(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Testing DDL deparsing support |
Date: | 2014-12-02 18:13:07 |
Message-ID: | 20141202181307.GO1737@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Robert Haas wrote:
> On Thu, Nov 27, 2014 at 11:43 PM, Ian Barwick <ian(at)2ndquadrant(dot)com> wrote:
> > A simple schedule to demonstrate this is available; execute from the
> > src/test/regress/ directory like this:
> >
> > ./pg_regress \
> > --temp-install=./tmp_check \
> > --top-builddir=../../.. \
> > --dlpath=. \
> > --schedule=./schedule_ddl_deparse_demo
>
> I haven't read the code, but this concept seems good to me.
Excellent, thanks.
> It has the unfortunate weakness that a difference could exist during
> the *middle* of the regression test run that is gone by the *end* of
> the run, but our existing pg_upgrade testing has the same weakness, so
> I guess we can view this as one more reason not to be too aggressive
> about having regression tests drop the unshared objects they create.
Agreed. Not dropping objects also helps test pg_dump itself; the normal
procedure there is run the regression tests, then pg_dump the regression
database. Objects that are dropped never exercise their corresponding
pg_dump support code, which I think is a bad thing. I think we should
institute a policy that regression tests must keep the objects they
create; maybe not all of them, but at least a sample large enough to
cover all interesting possibilities.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2014-12-02 18:13:52 | Re: Turning recovery.conf into GUCs |
Previous Message | Robert Haas | 2014-12-02 17:59:26 | Re: Add shutdown_at_recovery_target option to recovery.conf |