public inbox for [email protected]  

help / color / mirror / Atom feed

From: Andres Freund <[email protected]>
To: [email protected]
Subject: pgsql: instrumentation: Rename INSTR_TIME_LT macro to INSTR_TIME_GT
Date: Thu, 26 Feb 2026 16:14:17 +0000
Message-ID: <[email protected]> (raw)

instrumentation: Rename INSTR_TIME_LT macro to INSTR_TIME_GT

This was incorrectly named "LT" for "larger than" in e5a5e0a90750d66, but
that is against existing conventions, where "LT" means "less than".
Clarify by using "GT" for "greater than" in macro name, and add a missing
comment at the top of instr_time.h to note the macro's existence.

Reported by: Peter Smith <[email protected]>
Author: Lukas Fittl <[email protected]>
Reviewed-by: Andres Freund <[email protected]>
Discussion: https://postgr.es/m/CAHut%2BPut94CTpjQsqOJHdHkgJ2ZXq%2BqVSfMEcmDKLiWLW-hPfA%40mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/3218825271bd5ea3921ecd18e373bc76947c70bc

Modified Files
--------------
src/backend/executor/instrument.c    | 2 +-
src/include/portability/instr_time.h | 4 +++-
2 files changed, 4 insertions(+), 2 deletions(-)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: pgsql: instrumentation: Rename INSTR_TIME_LT macro to INSTR_TIME_GT
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox