From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Vik Fearing <vik(dot)fearing(at)dalibo(dot)com> |
Cc: | Josh Berkus <josh(at)agliodbs(dot)com>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, PG Hackers <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Subject: | Re: [PATCH] pg_sleep(interval) |
Date: | 2013-10-21 15:53:00 |
Message-ID: | 20131021155300.GJ2706@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Vik,
* Vik Fearing (vik(dot)fearing(at)dalibo(dot)com) wrote:
> A parser refactoring broke my code. I reported it and it was promptly
> fixed. When the fix came up for review, I said it needed a regression
> test to prevent it from happening again and I was told by the author
> that such a test would be "flimsy" and it went on to be committed (by
> that same guy) without one. I'm undecided whether I'll be contributing
> there any further.
To be fair, we've declined to add regression tests, and in cases even
removed them, if they have been a source of false positives or add a lot
of extra time to the regression suite without really adding a lot of
coverage. We continue to discuss, and need imv, more regression tests,
grouped into different sets which are run depending on the environment
and local configuration. Running the regression suite while doing quick
code iterations should be fast, while the build farm should run as many
regression tests as practical (which might depend on the system).
There has been some progress on this front recently by Peter E and
Andrew Dunstan, as I recall, but I've not followed it very closely.
They may very well already have this set up.
> The rigor here makes me want to try and try again.
This is certainly good to hear and I hope that you continue to
contribute and keep on trying.
Thanks!
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Boszormenyi Zoltan | 2013-10-21 15:57:51 | Re: Commitfest II CLosed |
Previous Message | Hannu Krosing | 2013-10-21 15:46:05 | Re: Commitfest II CLosed |