Re: psql - add SHOW_ALL_RESULTS option - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: psql - add SHOW_ALL_RESULTS option
Date
Msg-id CAFj8pRD6XuTgbLCaHra+t1PJBEm8SL+y-Qmhax3-5A=0Q9xO0g@mail.gmail.com
Whole thread Raw
In response to Re: psql - add SHOW_ALL_RESULTS option  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: psql - add SHOW_ALL_RESULTS option  (Fabien COELHO <coelho@cri.ensmp.fr>)
List pgsql-hackers
Hi

čt 22. 7. 2021 v 7:52 odesílatel Fabien COELHO <coelho@cri.ensmp.fr> napsal:

>>> The patch does not apply on Head anymore, could you rebase and post a
>>> patch. I'm changing the status to "Waiting for Author".
>>
>> 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.

Thank you! The patch update is in progress…

The newly added PSQL_WATCH_PAGER feature which broke the patch does not
seem to be tested anywhere, this is tiring:-(

Do you have any idea how this can be tested? It requires some pager that doesn't use blocking reading, and you need to do remote control of this pager. So it requires a really especially written pager just for this purpose. It is solvable, but I am not sure if it is adequate to this patch. 

Regards

Pavel



--
Fabien.

pgsql-hackers by date:

Previous
From: vignesh C
Date:
Subject: Re: Hook for extensible parsing.
Next
From: Ronan Dunklau
Date:
Subject: Re: ORDER BY pushdowns seem broken in postgres_fdw