pgsql: Don't use custom OID symbols in pg_type.dat, either.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Don't use custom OID symbols in pg_type.dat, either.
Date: 2020-10-29 17:33:48
Message-ID: E1kYBo4-0002Yi-Jc@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Don't use custom OID symbols in pg_type.dat, either.

On the same reasoning as in commit 36b931214, forbid using custom
oid_symbol macros in pg_type as well as pg_proc, so that we always
rely on the predictable macro names generated by genbki.pl.

We do continue to grant grandfather status to the names CASHOID and
LSNOID, although those are now considered deprecated aliases for the
preferred names MONEYOID and PG_LSNOID. This is because there's
likely to be client-side code using the old names, and this bout of
neatnik-ism doesn't quite seem worth breaking client code.

There might be a case for grandfathering EVTTRIGGEROID, too, since
externally-maintained PLs may reference that symbol. But renaming
such references to EVENT_TRIGGEROID doesn't seem like a particularly
heavy lift --- we make far more significant backend API changes in
every major release. For now I didn't add that, but we could
reconsider if there's pushback.

The other names changed here seem pretty unlikely to have any outside
uses. Again, we could add alias macros if there are complaints, but
for now I didn't.

As before, no need for a catversion bump.

John Naylor

Discussion: https://postgr.es/m/CAFBsxsHpCbjfoddNGpnnnY5pHwckWfiYkMYSF74PmP1su0+ZOw@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f90149e6285aaae6b48559afce1bd638ee26c33e

Modified Files
--------------
src/backend/bootstrap/bootstrap.c | 2 +-
src/backend/catalog/genbki.pl | 8 ++++++--
src/backend/commands/event_trigger.c | 2 +-
src/backend/utils/misc/pg_controldata.c | 10 +++++-----
src/fe_utils/print.c | 2 +-
src/include/catalog/pg_type.dat | 30 ++++++++++--------------------
src/include/catalog/pg_type.h | 7 +++++++
src/interfaces/ecpg/ecpglib/execute.c | 2 +-
src/pl/plperl/plperl.c | 4 ++--
src/pl/plpgsql/src/pl_comp.c | 2 +-
src/pl/plpgsql/src/pl_handler.c | 2 +-
src/pl/plpython/plpy_procedure.c | 2 +-
src/pl/tcl/pltcl.c | 2 +-
13 files changed, 38 insertions(+), 37 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2020-10-29 19:29:32 pgsql: Stabilize timetz test across DST transitions.
Previous Message Andres Freund 2020-10-29 04:54:23 pgsql: Fix wrong data table horizon computation during backend startup.