From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Stephen Frost <sfrost(at)snowman(dot)net> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |
Date: | 2017-03-20 14:46:38 |
Message-ID: | CA+Tgmoa5b5Xa-HM7x7aNayQKjhp17HxqbVNw0O6JEud8ZNmfew@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Mon, Mar 20, 2017 at 10:35 AM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:
> To be clear, I am not asking for any kind of special exception for
> myself.
>
> I continue to be of the opinion that this entire discussion is quite
> flipped from how we really should be running things- adding regression
> tests to improve code coverage, particularly when they're simply adding
> to the existing structure for those tests, should be strongly encouraged
> both before and after feature-freeze.
Any policy which permits a 3000 line code drop, whether to the
regression tests or otherwise, without prior discussion is, IMHO, a
very bad policy. It's not as if regression tests never break anything
or cause any problems. Do they need the same level of review as WARM
or rewriting the executor's expression evaluation? No. Does that
mean that they should totally bypass all of the review and discussion
that we do for other patches? No.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-03-20 14:54:46 | Re: Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |
Previous Message | Andres Freund | 2017-03-20 14:42:35 | Re: Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2017-03-20 14:47:53 | Re: Inadequate traces in TAP tests |
Previous Message | Andres Freund | 2017-03-20 14:42:35 | Re: Re: [COMMITTERS] pgsql: Improve pg_dump regression tests and code coverage |