Horton

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

Justin Pryzby <pryzby@telsasoft.com>

View thread View on postgr.es
From: Justin Pryzby <pryzby@telsasoft.com>
To: Greg Stark <stark@mit.edu>
Cc: Julien Rouhaud <rjuju123@gmail.com>, Ekaterina Sokolova <e.sokolova@postgrespro.ru>, Lukas Fittl <lukas@fittl.com>, pgsql-hackers@lists.postgresql.org
Date: 2022-03-28T23:28:32Z
Lists: pgsql-hackers 
> > +static void show_loop_info(Instrumentation *instrument, bool isworker,
> > +                          ExplainState *es);
> >
> > I think this should be done as a separate refactoring commit.

Right - the 0001 patch I sent seems independently beneficial, and makes the
changes in 0002 more apparent.  My 0001 could also be applied after the feature
freeze and before branching for v16..

Commits

  1. Fix wrong units in two ExplainPropertyFloat calls.