pgsql: Rename some toasting functions based on whether they are heap-sp - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Rename some toasting functions based on whether they are heap-sp
Date
Msg-id E1iGSKR-0008L6-55@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Rename some toasting functions based on whether they are heap-specific.

The old names for the attribute-detoasting functions names included
the word "heap," which seems outdated now that the heap is only one of
potentially many table access methods.

On the other hand, toast_insert_or_update and toast_delete are
heap-specific, so rename them by adding "heap_" as a prefix.

Not all of the work of making the TOAST system fully accessible to AMs
other than the heap is done yet, but there seems to be little harm in
getting this renaming out of the way now. Commit
8b94dab06617ef80a0901ab103ebd8754427ef5a already divided up the
functions among various files partially according to whether it was
intended that they should be heap-specific or AM-agnostic, so this is
just clarifying the division contemplated by that commit.

Patch by me, reviewed and tested by Prabhat Sabu, Thomas Munro,
Andres Freund, and Álvaro Herrera.

Discussion: http://postgr.es/m/CA+TgmoZv-=2iWM4jcw5ZhJeL18HF96+W1yJeYrnGMYdkFFnEpQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/2e8b6bfa90b252b1e1758364de7deff067d6058a

Modified Files
--------------
src/backend/access/common/detoast.c        | 26 +++++++++++++-------------
src/backend/access/common/indextuple.c     |  2 +-
src/backend/access/heap/heapam.c           |  8 ++++----
src/backend/access/heap/heaptoast.c        | 23 ++++++++++++-----------
src/backend/access/heap/rewriteheap.c      |  4 ++--
src/backend/access/table/toast_helper.c    |  4 ++--
src/backend/executor/tstoreReceiver.c      |  2 +-
src/backend/storage/large_object/inv_api.c |  2 +-
src/backend/utils/adt/expandedrecord.c     |  4 ++--
src/backend/utils/fmgr/fmgr.c              |  8 ++++----
src/include/access/detoast.h               | 16 ++++++++--------
src/include/access/heaptoast.h             | 12 ++++++------
src/pl/plpgsql/src/pl_exec.c               |  2 +-
src/test/regress/regress.c                 |  2 +-
14 files changed, 58 insertions(+), 57 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix bitshiftright()'s zero-padding some more.
Next
From: Andrew Dunstan
Date:
Subject: pgsql: Handle spaces in OpenSSL install location for MSVC