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