From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |
Date: | 2017-03-20 13:11:52 |
Message-ID: | 20170320131152.GG9812@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Peter,
* Peter Eisentraut (peter(dot)eisentraut(at)2ndquadrant(dot)com) wrote:
> On 3/20/17 08:33, Stephen Frost wrote:
> >> So was this 3340 line patch posted or discussed anyplace before it got
> >> committed?
> > I've mentioned a few times that I'm working on improving pg_dump
> > regression tests and code coverage, which is what these were. I'm a bit
> > surprised that it's, apparently, a surprise to anyone or that strictly
> > adding regression tests in the existing framework deserves very much
> > discussion.
>
> I think we had this discussion about adding a large number of (pg_dump)
> tests without discussion or review already about a year ago, so I for
> one am surprised that are still surprised.
The concern you raised at the time, from my recollection, was that I had
added a new set of TAP tests post feature-freeze for pg_dump and there
was concern that it might cause an issue for packagers.
I don't recall a concern being raised about the tests themselves, and I
intentionally worked to make sure this landed before feature-freeze to
avoid that issue, though I believe we should actually be looking to try
to add tests post feature-freeze too, as we work to test things prior to
the release.
Thanks!
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2017-03-20 13:30:46 | Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |
Previous Message | Peter Eisentraut | 2017-03-20 13:07:39 | Re: Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2017-03-20 13:12:05 | Inadequate traces in TAP tests |
Previous Message | Peter Eisentraut | 2017-03-20 13:07:39 | Re: Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |