Re: psql - add SHOW_ALL_RESULTS option

Hi

čt 22. 7. 2021 v 7:52 odesílatel Fabien COELHO <coelho(at)cri(dot)ensmp(dot)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.