public inbox for [email protected]
help / color / mirror / Atom feedFrom: Peter Eisentraut <[email protected]>
To: Fabien COELHO <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: psql - add SHOW_ALL_RESULTS option
Date: Fri, 24 Sep 2021 14:42:33 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <alpine.DEB.2.22.394.2107221655020.25741@pseudo>
References: <[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<alpine.DEB.2.22.394.2104151336060.191178@pseudo>
<[email protected]>
<alpine.DEB.2.22.394.2106121130400.4105466@pseudo>
<CALDaNm0+pzGgp-M1mAoUVYRtjadbWYuB96J8RaV9iH3+HA8FTA@mail.gmail.com>
<alpine.DEB.2.22.394.2107151742300.1705285@pseudo>
<[email protected]>
<alpine.DEB.2.22.394.2107221655020.25741@pseudo>
On 22.07.21 16:58, Fabien COELHO wrote:
>>> Ok. I noticed. The patch got significantly broken by the watch pager
>>> commit. I also have to enhance the added tests (per Peter request).
>>
>> I wrote a test to check psql query cancel support. I checked that it
>> fails against the patch that was reverted. Maybe this is useful.
>
> Here is the updated version (v8? I'm not sure what the right count is),
> which works for me and for "make check", including some tests added for
> uncovered paths.
I was looking at adding test coverage for the issue complained about in
[0]. That message said that the autocommit logic was broken, but
actually the issue was with the ON_ERROR_ROLLBACK logic. However, it
turned out that neither feature had any test coverage, and they are
easily testable using the pg_regress setup, so I wrote tests for both
and another little thing I found nearby.
It turns out that your v8 patch still has the issue complained about in
[0]. The issue is that after COMMIT AND CHAIN, the internal savepoint
is gone, but the patched psql still thinks it should be there and tries
to release it, which leads to errors.
[0]: https://www.postgresql.org/message-id/[email protected]
view thread (104+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: psql - add SHOW_ALL_RESULTS option
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox