pgsql: amcheck: Reword some messages and fix an alignment problem. - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: amcheck: Reword some messages and fix an alignment problem.
Date
Msg-id [email protected]
Whole thread Raw
List 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/[email protected]

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(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: pgsql: Improve quoting in some error messages
Next
From: Tom Lane
Date:
Subject: pgsql: Fix obsolete comments referencing JoinPathExtraData.extra_latera