Rename of varatt_external to varatt_external_oid

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de>
Subject: Rename of varatt_external to varatt_external_oid
Date: 2026-09-03 23:33:41
Message-ID: apoD1Ulet2PtDrRV@paquier.xyz
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi all,
(Tom and Andres in CC, as they've commented on the original "round 1"
thread)

In order to add support of 8-byte TOAST values, one item that I have
on my list of items is a set of renames in the varatt.h definitions
for some objects, as of:
- varatt_external -> varatt_external_oid
- VARTAG_ONDISK -> VARTAG_ONDISK_OID
- TOAST_POINTER_SIZE -> TOAST_OID_POINTER_SIZE
- TOAST_MAX_CHUNK_SIZE -> TOAST_OID_MAX_CHUNK_SIZE
- Few macros in varatt.h.
- (I'm aware of the control file bit, left out on purpose as its
tracking is still valid even if we extend to 8 bytes.)

This is related to the following, larger patch set, but I wanted to
make a last call before proceeding as it is hidden in a much larger
set of patches:
https://www.postgresql.org/message-id/flat/af19kUjwjhaoUTLn(at)paquier(dot)xyz

Tom has commented about this part of the patch here, presented in
0003:
https://www.postgresql.org/message-id/1891064.1754681536@sss.pgh.pa.us

This is a more ambitious version of it, with more mechanical renames
to make the difference between the OID and OID8 flavors of the
pointers.

One point of the renames is to make extension folks aware of the fact
that the new TOAST structures may need handling due to the new 8-byte
values. Still, I also feel guilty to not provide a set of
compatibility definitions, which is of course one option. Note that
I'm planning to do all that early in the release cycle for v20, to get
room for it.

Rebased on HEAD is the patch I have for staging. Any thoughts,
comments or objections?
--
Michael

Attachment Content-Type Size
0001-Rename-varatt_external-to-varatt_external_oid.patch text/plain 27.3 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Paul Kim 2026-09-04 00:04:47 Re: Be strict when request to flush past end of WAL in WaitXLogInsertionsToFinish
Previous Message Robert Haas 2026-09-03 23:19:59 Re: WAIT FOR NO_THROW option could use some documentation