You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Andres Freund f495b65a56 Fix potential assertion failure when reindexing a pg_class index. ... When reindexing individual indexes on pg_class it was possible to either trigger an assertion failure: TRAP: FailedAssertion("!(!ReindexIsProcessingIndex(((index)->rd_id))) That's because reindex_index() called SetReindexProcessing() - which enables an asserts ensuring no index insertions happen into the index - before calling RelationSetNewRelfilenode(). That not correct for indexes on pg_class, because RelationSetNewRelfilenode() updates the relevant pg_class row, which needs to update the indexes. The are two reasons this wasn't noticed earlier. Firstly the bug doesn't trigger when reindexing all of pg_class, as reindex_relation has code "hiding" all yet-to-be-reindexed indexes. Secondly, the bug only triggers when the the update to pg_class doesn't turn out to be a HOT update - otherwise there's no index insertion to trigger the bug. Most of the time there's enough space, making this bug hard to trigger. To fix, move RelationSetNewRelfilenode() to before the SetReindexProcessing() (and, together with some other code, to outside of the PG_TRY()). To make sure the error checking intended by SetReindexProcessing() is more robust, modify CatalogIndexInsert() to check ReindexIsProcessingIndex() even when the update is a HOT update. Also add a few regression tests for REINDEXing of system catalogs. The last two improvements would have prevented some of the issues fixed in 5c1560606d from being introduced in the first place. Reported-By: Michael Paquier Diagnosed-By: Tom Lane and Andres Freund Author: Andres Freund Reviewed-By: Tom Lane Discussion: https://postgr.es/m/20190418011430.GA19133@paquier.xyz Backpatch: 9.4-, the bug is present in all branches		7 years ago
..
binary.sql	…
ddl.sql	Relax overly strict assertion	7 years ago
decoding_in_xact.sql	…
decoding_into_rel.sql	Fix crash when logical decoding is invoked from a PL function.	8 years ago
messages.sql	…
permissions.sql	…
prepared.sql	…
replorigin.sql	Fix more user-visible elog() calls.	8 years ago
rewrite.sql	Fix potential assertion failure when reindexing a pg_class index.	7 years ago
slot.sql	Force synchronous commit in new-ish test_decoding test.	9 years ago
spill.sql	Make contrib/test_decoding regression tests safe for CZ locale.	9 years ago
time.sql	…
toast.sql	Add regression test for wide REPLICA IDENTITY FULL updates.	9 years ago
xact.sql	…