On Sat, Mar 03, 2018 at 08:58:18AM -0500, Peter Eisentraut wrote: > On 3/2/18 03:23, Michael Paquier wrote: >> That's a better idea. I have reworked that in 0001. What do you think? >> This has the same effect as diag(), which shows information directly to >> the screen, and that's the behavior I was looking for. > > I ended up using plan skip_all, which seems to be intended for this > purpose, according to the documentation.
Yes, that's what I got as well first but... >> I have concluded about using the most simple name: PG_TEST_EXTRA. A >> documentation attempt is added as well. This is unfortunately close to >> EXTRA_TESTS. It would be tempting to directly reuse EXTRA_TESTS as >> well, however this is used only for the core regression tests now, so a >> separated variable seems adapted to me. > > Committed. Very nice. Thanks for the commit, Peter, PG_TEST_EXTRA is already set is my environment. -- Michael
signature.asc
Description: PGP signature
