On Tue, Sep 06, 2022 at 11:37:32AM +0500, Ibrar Ahmed wrote:
> The patch failed different regression tests on all platforms. Please
> correct that and send an updated patch.
>
> [06:40:02.370] Test Summary Report
> [06:40:02.370] ---
> [06:40:02.370] t/002_pg_upgrade.pl (Wstat: 256
;
> pgsql-hackers@lists.postgresql.org>; Zhenghua Lyu
> *Subject:* Re: Summary Sort workers Stats in EXPLAIN ANALYZE
>
> ⚠ External Email
>
> Hi,
>
> On Thu, Mar 24, 2022 at 07:50:11AM +, Jian Guo wrote:
> > For a simple demo, with this explain statement：
> &
lease review, thanks.
From: Julien Rouhaud
Sent: Friday, March 25, 2022 17:04
To: Jian Guo
Cc: pgsql-hackers@lists.postgresql.org ;
Zhenghua Lyu
Subject: Re: Summary Sort workers Stats in EXPLAIN ANALYZE
⚠ External Email
Hi,
On Thu, Mar 24, 2022 at 07:50:11AM
On Fri, Mar 25, 2022 at 05:04:53PM +0800, Julien Rouhaud wrote:
> I think the idea is interesting, however there are a few problems in the
> patch.
>
> First, I think that it should only be done in the VERBOSE OFF mode. If you
> ask
> for a VERBOSE output you don't need both the details and the
Hi,
On Thu, Mar 24, 2022 at 07:50:11AM +, Jian Guo wrote:
> For a simple demo, with this explain statement：
>
> -- Test sort stats summary
> set force_parallel_mode=on;
> select explain_filter('explain (analyze, summary off, timing off, costs off,
> format json) select * from tenk1 order by u
e", +
"Average Sort Space Used": N, +
"Peak Sort Space Used": N, +
"Sort Space Type": "Disk", +
From: Jian Guo
Sent: Monday, March 21, 2022 15:50
To: pgsql-hackers@lists
There is some problem with the last patch, I have removed the
`ExplainOpenWorker` call to fix.
And also, I have added a test case in explain.sql according to the code change.
From: Jian Guo
Sent: Monday, March 21, 2022 11:36
To: pgsql-hackers@lists.postgresql.or
