From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Noah Misch <noah(at)leadboat(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: TAP test breakage on MacOS X |
Date: | 2014-10-30 23:47:00 |
Message-ID: | 20141030234700.GN17724@awork2.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2014-10-30 19:30:04 -0400, Tom Lane wrote:
> Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com> writes:
> > If our policy is that tests are there primarily for developers then I agree with you.
>
> > If not, then would we be OK with make check being a no-op unless you'd configured with --enable-make-check?
>
> > Making this something you have to enable will seriously limit the number of people running the TAP tests, simply because few will know to enable them.
>
> Well, TBH I have no problem with that at the moment, because as Robert has
> pointed out the current TAP tests are of close to zero value. The odds
> that they'll find anything in the hands of Joe Random User are far lower
> than the odds that they'll break Joe's build.
I already have a couple more ready once this has stabilized...
I personally don't agree that they have no value at this point. At least
the pg_basebackup tests test paths that are not executed *at all*
otherwise and which are not trivial. To my knowledge it's the only thing
in our tests that exercises walsender and wal_level != minimal.
> At some point down the road that value judgement might (hopefully will)
> reverse, and then we could deal with it by making --enable-tap-tests the
> default. But even then there would be a place for
> --disable-tap-tests.
Which would be what exactly?
> The current situation, where the only way to disable the TAP tests is to
> not do make check-world, is utterly unacceptable given their low present
> usefulness and lack of proven portability.
Agreed.
> I opined before that we should rip those tests out of the 9.4 branch
> altogether. I'm willing to leave them there if we have an --enable
> switch controlling them, though.
Hm. I'm not convinced that that's the best way, but I'm not going fight
hard.
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-10-30 23:53:54 | Re: TAP test breakage on MacOS X |
Previous Message | Andres Freund | 2014-10-30 23:34:52 | Re: proposal: CREATE DATABASE vs. (partial) CHECKPOINT |