Horton

Re: wrong units in ExplainPropertyFloat

Tom Lane <tgl@sss.pgh.pa.us>

View thread View on postgr.es
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Justin Pryzby <pryzby@telsasoft.com>
Cc: Andres Freund <andres@anarazel.de>, pgsql-hackers@postgresql.org
Date: 2021-04-16T15:32:01Z
Lists: pgsql-hackers 
Justin Pryzby <pryzby@telsasoft.com> writes:
> Text mode uses appendStringInfo() for high density output, so this only affects
> non-text output, but it turns out that units aren't shown for nontext format
> anyway - this seems like a deficiency, but it means there's no visible bug.

Yeah, I concur: these should say "ms", but it's only latent so it's
not surprising nobody's noticed.  Pushed.

			regards, tom lane

Commits

  1. Fix wrong units in two ExplainPropertyFloat calls.

  2. Add 'unit' parameter to ExplainProperty{Integer,Float}.