Horton

Re: [PATCH] Add extra statistics to explain for Nested Loop

hubert depesz lubaczewski <depesz@depesz.com>

View thread View on postgr.es
From: hubert depesz lubaczewski <depesz@depesz.com>
To: Julien Rouhaud <rjuju123@gmail.com>
Cc: Pavel Stehule <pavel.stehule@gmail.com>, Pierre Giraud <pierre.giraud@dalibo.com>, Anastasia Lubennikova <a.lubennikova@postgrespro.ru>, e.sokolova@postgrespro.ru, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2020-10-17T08:23:45Z
Lists: pgsql-hackers 
On Sat, Oct 17, 2020 at 12:26:08PM +0800, Julien Rouhaud wrote:
> >> -               ->  Nested Loop (actual rows=N loops=N)
> >> +              ->  Nested Loop (actual min_rows=0 rows=0 max_rows=0 loops=2)
> > This interface is ok - there is not too much space for creativity.
> Yes I also think it's ok. We should also consider usability for tools
> like explain.depesz.com, I don't know if the current output is best.
> I'm adding Depesz and Pierre which are both working on this kind of
> tool for additional input.

Thanks for heads up. This definitely will need some changes on my side,
but should be easy to handle.

Best regards,

depesz

Commits

  1. Fix wrong units in two ExplainPropertyFloat calls.