pgsql: Mostly-cosmetic adjustments of TOAST-related macros.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Mostly-cosmetic adjustments of TOAST-related macros.
Date: 2021-03-22 18:01:31
Message-ID: E1lOOrr-0005pi-8J@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Mostly-cosmetic adjustments of TOAST-related macros.

The authors of bbe0a81db hadn't quite got the idea that macros named
like SOMETHING_4B_C were only meant for internal endianness-related
details in postgres.h. Choose more legible names for macros that are
intended to be used elsewhere. Rearrange postgres.h a bit to clarify
the separation between those internal macros and ones intended for
wider use.

Also, avoid using the term "rawsize" for true decompressed size;
we've used "extsize" for that, because "rawsize" generally denotes
total Datum size including header. This choice seemed particularly
unfortunate in tests that were comparing one of these meanings to
the other.

This patch includes a couple of not-purely-cosmetic changes: be
sure that the shifts aligning compression methods are unsigned
(not critical today, but will be when compression method 2 exists),
and fix broken definition of VARATT_EXTERNAL_GET_COMPRESSION (now
VARATT_EXTERNAL_GET_COMPRESS_METHOD), whose callers worked only
accidentally.

Discussion: https://postgr.es/m/574197.1616428079@sss.pgh.pa.us

Branch
------
master

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

Modified Files
--------------
src/backend/access/common/detoast.c | 4 +-
src/backend/access/common/toast_compression.c | 40 +++++-----
src/backend/access/common/toast_internals.c | 8 +-
src/include/access/toast_internals.h | 13 ++--
src/include/postgres.h | 106 +++++++++++++-------------
5 files changed, 87 insertions(+), 84 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tomas Vondra 2021-03-22 23:55:22 pgsql: Optimize allocations in bringetbitmap
Previous Message Tom Lane 2021-03-22 15:20:54 pgsql: Remove useless configure probe for <lz4/lz4.h>.