On 14.03.21 10:54, Fabien COELHO wrote:
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.
Committed. The last thing I fixed was the diff in the copy2.out
regression test. The order of the notices with respect to the error
messages was wrong. I fixed that by switching back to the regular
notice processor during COPY handling.
Btw., not sure if that was mentioned before, but a cool use of this is
to \watch multiple queries at once, like
select current_date \; select current_time \watch