Re: [PATCH] Add extra statistics to explain for Nested Loop - Mailing list pgsql-hackers

From Michael Christofides
Subject Re: [PATCH] Add extra statistics to explain for Nested Loop
Date
Msg-id CAFwT4nDbor4JMkH1_wocbmvt-U39V0mgrWY9dXDk_mP6Hi-uyg@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Add extra statistics to explain for Nested Loop  ("David G. Johnston" <david.g.johnston@gmail.com>)
List pgsql-hackers
> New version of this patch prints extra statistics for all cases of
> multiple loops, not only for Nested Loop. Also I fixed the example by
> adding VERBOSE.
>
> Please don't hesitate to share any thoughts on this topic!

Thanks a lot for working on this! I really like the extra details, and
including it only with VERBOSE sounds good.

> rows * loops is still an important calculation.
>
> Why not just add total_rows while we are at it - last in the listing?
>
> (actual rows=N loops=N min_rows=N max_rows=N total_rows=N)

This total_rows idea from David would really help us too, especially
in the cases where the actual rows is rounded down to zero. We make an
explain visualisation tool, and it'd be nice to show people a better
total than loops * actual rows. It would also help the accuracy of
some of our tips, that use this number.

Apologies if this input is too late to be helpful.

Cheers,
Michael



pgsql-hackers by date:

Previous
From: Ashutosh Bapat
Date:
Subject: Re: search_plan_tree(): handling of non-leaf CustomScanState nodes causes segfault
Next
From: Bharath Rupireddy
Date:
Subject: Re: Narrow the scope of the variable outputstr in logicalrep_write_tuple