Re: psql - add SHOW_ALL_RESULTS option

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


Hello Peter,

>> This reply was two months ago, and nothing has happened, so I have
>> marked the patch as RwF.
>
> Given the ongoing work on returning multiple result sets from stored
> procedures[0], I went to dust off this patch.
>
> Based on the feedback, I put back the titular SHOW_ALL_RESULTS option,
> but set the default to on. I fixed the test failure in
> 013_crash_restart.pl. I also trimmed back the test changes a bit so
> that the resulting test output changes are visible better. (We could
> make those stylistic changes separately in another patch.) I'll put
> this back into the commitfest for another look.

Thanks a lot for the fixes and pushing it forward!

My 0.02€: I tested this updated version and do not have any comment on
this version. From my point of view it could be committed. I would not
bother to separate the test style ajustments.

--
Fabien.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Fabien COELHO 2021-03-14 10:17:56 Re: pgbench - add pseudo-random permutation function
Previous Message Michael Paquier 2021-03-14 08:59:59 Re: PITR promote bug: Checkpointer writes to older timeline