pgsql: amcheck: Reword some messages and fix an alignment problem.

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: amcheck: Reword some messages and fix an alignment problem.
Date: 2021-04-14 17:17:28
Message-ID: E1lWj8q-0001fV-3c@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

amcheck: Reword some messages and fix an alignment problem.

We don't need to mention the attribute number in these messages, because
there's a dedicated column for that, but we should mention the toast
value ID, because that's really useful for any follow-up troubleshooting
the user wants to do. This also rewords some of the messages to hopefully
read a little better.

Also, use VARATT_EXTERNAL_GET_POINTER in case we're accessing a TOAST
pointer that isn't aligned on a platform that's fussy about alignment,
so that we don't crash while corruption-checking the user's data.

Mark Dilger, reviewed by me.

Discussion: http://postgr.es/m/7D3B9BF6-50D0-4C30-8506-1C1851C7F96F@enterprisedb.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/9acaf1a62197205b06a85afbfcaa7ffaac939ef3

Modified Files
--------------
contrib/amcheck/verify_heapam.c | 63 ++++++++++++++++++-------------
src/bin/pg_amcheck/t/004_verify_heapam.pl | 4 +-
2 files changed, 38 insertions(+), 29 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2021-04-14 18:28:32 pgsql: Fix obsolete comments referencing JoinPathExtraData.extra_latera
Previous Message Peter Eisentraut 2021-04-14 07:18:42 pgsql: Improve quoting in some error messages