Re: explain analyze rows=%.0f
Daniel Gustafsson <daniel@yesql.se>
From: Daniel Gustafsson <daniel@yesql.se>
To: Ibrar Ahmed <ibrar.ahmad@gmail.com>
Cc: "Gregory Stark (as CFM)" <stark.cfm@gmail.com>,
Tom Lane <tgl@sss.pgh.pa.us>,
Robert Haas <robertmhaas@gmail.com>,
Amit Kapila <amit.kapila16@gmail.com>,
Justin Pryzby <pryzby@telsasoft.com>,
Peter Geoghegan <pg@bowt.ie>,
vignesh C <vignesh21@gmail.com>,
"David G. Johnston" <david.g.johnston@gmail.com>,
pgsql-hackers@lists.postgresql.org
Date: 2023-08-01T20:29:05Z
Lists: pgsql-hackers
> On 3 Jul 2023, at 18:34, Daniel Gustafsson <daniel@yesql.se> wrote: > >> On 8 Jun 2023, at 19:49, Ibrar Ahmed <ibrar.ahmad@gmail.com> wrote: >> On Mon, Mar 20, 2023 at 7:56 PM Gregory Stark (as CFM) <stark.cfm@gmail.com <mailto:stark.cfm@gmail.com>> wrote: > >> This patch was marked Returned with Feedback and then later Waiting on >> Author. And it hasn't had any updates since January. What is the state >> on this feedback? If it's already done we can set the patch to Ready >> for Commit and if not do you disagree with the proposed changes? >> >> If there is a consensus to modify the test cases' output, I am willing to >> make the necessary changes and adjust the patch accordingly. However, >> if there is a preference to keep the output of certain test cases unchanged, >> I can rebase and modify the patch accordingly to accommodate those preferences. > > As there hasn't been any other comments I suggest updating your patch to > address Tom's comments to see if we can make progress here. Since there hasn't been any updates here, and the thread has been stalled, I'm marking this returned with feedback. Please feel free to resubmit a version of the patch addressing comments to a future CF. -- Daniel Gustafsson
Commits
-
EXPLAIN: Always use two fractional digits for row counts.
- 95dbd827f2ed 18.0 landed
-
Adjust EXPLAIN test case to filter out "Actual Rows" values.
- 44cbba9a7f51 18.0 cited
-
Allow EXPLAIN to indicate fractional rows.
- ddb17e387aa2 18.0 landed
-
Fix pgbench performance issue induced by commit af35fe501.
- fb056564ec5b 18.0 cited