Mostly-cosmetic adjustments of TOAST-related macros.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 22 Mar 2021 17:43:10 +0000 (13:43 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 22 Mar 2021 17:43:10 +0000 (13:43 -0400)
commitaeb1631ed207cef2d80e20f79eb52c72f03bca7d
tree5dd9966c424a78842a6ab0d6960ecc7fb6424710
parent2c75f8a612b207c7d36e5dc73317dc9ab6fb29d4
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
src/backend/access/common/detoast.c
src/backend/access/common/toast_compression.c
src/backend/access/common/toast_internals.c
src/include/access/toast_internals.h
src/include/postgres.h