From: | Julien Rouhaud <rjuju123(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Justin Pryzby <pryzby(at)telsasoft(dot)com>, pgsql-hackers(at)postgresql(dot)org, Robert Haas <rhaas(at)postgresql(dot)org>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Subject: | Re: default to to ON_ERROR_STOP=on (Re: psql: exit status with multiple -c and -f) |
Date: | 2022-01-17 00:10:43 |
Message-ID: | 20220117001043.hgjci4zqt3l4susa@jrouhaud |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On Mon, Dec 27, 2021 at 12:31:07PM -0500, Tom Lane wrote:
> Justin Pryzby <pryzby(at)telsasoft(dot)com> writes:
> > I think the current behavior of the regression test SQL scripts is exactly the
> > opposite of what's desirable for almost all other scripts. The attached makes
> > ON_ERROR_STOP the default, and runs the regression tests with ON_ERROR_STOP=0.
>
> > Is it viable to consider changing this ?
>
> I don't think so. The number of scripts you will break is far greater
> than the number whose behavior will be improved, because people who
> wanted this behavior will already be selecting it. Maybe this wasn't
> the greatest choice of default, but it's about twenty years too late
> to change it.
>
> I'd also note that I see a fairly direct parallel to "set -e" in
> shell scripts, which is likewise not the default.
>
> We could consider documentation changes to make this issue
> more visible, perhaps. Not sure what would be a good place.
I'm marking the CF entry as returned with feedback as it's been a few weeks
without proposal for documentation change.
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2022-01-17 00:28:03 | Time to increase hash_mem_multiplier default? |
Previous Message | Tomas Vondra | 2022-01-16 23:44:50 | Re: extended stats on partitioned tables |