Re: psql - add SHOW_ALL_RESULTS option

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>
Cc: "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: psql - add SHOW_ALL_RESULTS option
Date: 2021-09-25 09:28:52
Message-ID: alpine.DEB.2.22.394.2109251058000.3216312@pseudo
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Hallo Peter,

> 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.

Indeed. Thanks for the catch.

Attached v9 integrates your tests and makes them work.

--
Fabien.

Attachment Content-Type Size
psql-show-all-results-9.patch text/x-diff 53.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2021-09-25 09:55:03 Re: OpenSSL 3.0.0 compatibility
Previous Message vignesh C 2021-09-25 07:45:08 Re: Column Filtering in Logical Replication