Hi, On 2023-01-20 22:27:07 -0500, Tom Lane wrote: > Andres Freund <and...@anarazel.de> writes: > >> Perhaps an INSTR_TIME_ZERO() that could be assigned in variable definitions > >> could give us the best of both worlds? > > > I tried that in the attached 0005. I found that it reads better if I also > > add > > INSTR_TIME_CURRENT(). If we decide to go for this, I'd roll it into 0001 > > instead, but I wanted to get agreement on it first. > > -1 from here. This forecloses the possibility that it's best to use more > than one assignment to initialize the value, and the code doesn't read > any better than it did before.
I think it does read a bit better, but it's a pretty small improvement. So I'll leave this aspect be for now. Thanks for checking. Greetings, Andres Freund
