pgsql: Restore lost amcheck TOAST test coverage.

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Restore lost amcheck TOAST test coverage.
Date: 2020-07-31 22:34:43
Message-ID: E1k1dbv-0007sK-6T@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Restore lost amcheck TOAST test coverage.

Commit eba77534 fixed an amcheck false positive bug involving
inconsistencies in TOAST input state between table and index. A test
case was added that verified that such an inconsistency didn't result in
a spurious corruption related error.

Test coverage from the test was accidentally lost by commit 501e41dd,
which propagated ALTER TABLE ... SET STORAGE attstorage state to
indexes. This broke the test because the test specifically relied on
attstorage not being propagated. This artificially forced there to be
index tuples whose datums were equivalent to the datums in the heap
without the datums actually being bitwise equal.

Fix this by updating pg_attribute directly instead. Commit 501e41dd
made similar changes to a test_decoding TOAST-related test case which
made the same assumption, but overlooked the amcheck test case.

Backpatch: 11-, just like commit eba77534 (and commit 501e41dd).

Branch
------
REL_11_STABLE

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

Modified Files
--------------
contrib/amcheck/expected/check_btree.out | 16 ++++++++++++----
contrib/amcheck/sql/check_btree.sql | 14 ++++++++++----
2 files changed, 22 insertions(+), 8 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2020-08-01 00:16:26 pgsql: Improve programmer docs for simplehash and dynahash.
Previous Message Tom Lane 2020-07-31 21:11:39 pgsql: Fix oversight in ALTER TYPE: typmodin/typmodout must propagate t