Hi,
On 1/21/23 05:14, Andres Freund wrote:
> The elapsed time is already inherently unstable, so we shouldn't have any test
> output showing the time.
>
> But I doubt showing it in every explain is a good idea - we use instr_time in
> plenty of other places. Why show it in explain, but not in all those other
> places?
Yeah. I thought it would only be an issue if we showed it
unconditionally in EXPLAIN ANALYZE. If we only show it with TIMING ON,
we're likely fine with pretty much all regression tests.
But given the different opinions, I'll leave it out in the new patch set
for the moment being.
--
David Geier
(ServiceNow)
