From d92eb70562c4736febc24f164f6d43fd6324f680 Mon Sep 17 00:00:00 2001 From: Richard Guo Date: Tue, 1 Sep 2026 17:02:03 +0900 Subject: [PATCH v1] Fix OID-counter-sensitive row ordering in foreign_key test Several queries in the foreign_key test sort pg_constraint rows with "ORDER BY oid::regclass::text". These OIDs are constraint OIDs, not relation OIDs, so the regclass cast renders them as numeric strings, and text comparison of numeric strings depends on digit count. If the OID counter crosses a power of ten while the test runs, the output order flips, causing spurious test failures. This cannot happen in a fresh cluster, but installcheck against a long-lived cluster can hit it. Sort by plain oid instead, which compares numerically and preserves the intended creation order. The expected row orders do not change. --- src/test/regress/expected/foreign_key.out | 16 ++++++++-------- src/test/regress/sql/foreign_key.sql | 16 ++++++++-------- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/src/test/regress/expected/foreign_key.out b/src/test/regress/expected/foreign_key.out index ac044eb40fa..ddd138551bd 100644 --- a/src/test/regress/expected/foreign_key.out +++ b/src/test/regress/expected/foreign_key.out @@ -1936,7 +1936,7 @@ ALTER TABLE fk_partitioned_fk_1 ADD FOREIGN KEY (a, b) REFERENCES fk_notpartitio ALTER TABLE fk_partitioned_fk ATTACH PARTITION fk_partitioned_fk_1 FOR VALUES FROM (0,0) TO (1000,1000); -- Child constraint will remain valid. SELECT conname, convalidated, conrelid::regclass FROM pg_constraint -WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass::text; +WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid; conname | convalidated | conrelid ------------------------------+--------------+--------------------- fk_partitioned_fk_a_b_fkey | f | fk_partitioned_fk @@ -1947,7 +1947,7 @@ WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass: ALTER TABLE fk_partitioned_fk VALIDATE CONSTRAINT fk_partitioned_fk_a_b_fkey; -- All constraints are now valid. SELECT conname, convalidated, conrelid::regclass FROM pg_constraint -WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass::text; +WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid; conname | convalidated | conrelid ------------------------------+--------------+--------------------- fk_partitioned_fk_a_b_fkey | t | fk_partitioned_fk @@ -1967,7 +1967,7 @@ TRUNCATE fk_partitioned_fk_2; ALTER TABLE fk_partitioned_fk ATTACH PARTITION fk_partitioned_fk_2 FOR VALUES FROM (1000,1000) TO (2000,2000); -- The child constraint will also be valid. SELECT conname, convalidated FROM pg_constraint -WHERE conrelid = 'fk_partitioned_fk_2'::regclass ORDER BY oid::regclass::text; +WHERE conrelid = 'fk_partitioned_fk_2'::regclass ORDER BY oid; conname | convalidated ------------------------------+-------------- fk_partitioned_fk_2_a_b_fkey | t @@ -1984,7 +1984,7 @@ ALTER TABLE fk_partitioned_fk_3 ATTACH PARTITION fk_partitioned_fk_3_1 FOR VALUE ALTER TABLE fk_partitioned_fk ATTACH PARTITION fk_partitioned_fk_3 FOR VALUES FROM (2000,2000) TO (3000,3000); -- All constraints are now valid. SELECT conname, convalidated, conrelid::regclass FROM pg_constraint -WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass::text; +WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid; conname | convalidated | conrelid --------------------------------+--------------+----------------------- fk_partitioned_fk_a_b_fkey | t | fk_partitioned_fk @@ -2009,7 +2009,7 @@ ALTER TABLE fk_notpartitioned_fk ADD CONSTRAINT fk_notpartitioned_fk_a_b_fkey2 FOREIGN KEY (a, b) REFERENCES fk_partitioned_pk NOT ENFORCED; -- All constraints will be invalid, and _fkey2 constraints will not be enforced. SELECT conname, conenforced, convalidated FROM pg_constraint -WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid::regclass::text; +WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid; conname | conenforced | convalidated ----------------------------------+-------------+-------------- fk_notpartitioned_fk_a_b_fkey | t | f @@ -2024,7 +2024,7 @@ ALTER TABLE fk_notpartitioned_fk VALIDATE CONSTRAINT fk_notpartitioned_fk_a_b_fk ALTER TABLE fk_notpartitioned_fk ALTER CONSTRAINT fk_notpartitioned_fk_a_b_fkey2 ENFORCED; -- All constraints are now valid and enforced. SELECT conname, conenforced, convalidated FROM pg_constraint -WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid::regclass::text; +WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid; conname | conenforced | convalidated ----------------------------------+-------------+-------------- fk_notpartitioned_fk_a_b_fkey | t | t @@ -2042,7 +2042,7 @@ CREATE TABLE fk_partitioned_pk_3 PARTITION OF fk_partitioned_pk FOR VALUES FROM CREATE TABLE fk_partitioned_pk_3_1 PARTITION OF fk_partitioned_pk_3 FOR VALUES FROM (2000) TO (2100); SELECT conname, conenforced, convalidated FROM pg_constraint WHERE conrelid = 'fk_partitioned_pk'::regclass AND contype = 'f' -ORDER BY oid::regclass::text; +ORDER BY oid; conname | conenforced | convalidated ------------+-------------+-------------- selffk | t | f @@ -2056,7 +2056,7 @@ ALTER TABLE fk_partitioned_pk_2 VALIDATE CONSTRAINT selffk; ALTER TABLE fk_partitioned_pk VALIDATE CONSTRAINT selffk; SELECT conname, conenforced, convalidated FROM pg_constraint WHERE conrelid = 'fk_partitioned_pk'::regclass AND contype = 'f' -ORDER BY oid::regclass::text; +ORDER BY oid; conname | conenforced | convalidated ------------+-------------+-------------- selffk | t | t diff --git a/src/test/regress/sql/foreign_key.sql b/src/test/regress/sql/foreign_key.sql index a93e81b42bc..7afa37c8025 100644 --- a/src/test/regress/sql/foreign_key.sql +++ b/src/test/regress/sql/foreign_key.sql @@ -1449,14 +1449,14 @@ ALTER TABLE fk_partitioned_fk ATTACH PARTITION fk_partitioned_fk_1 FOR VALUES FR -- Child constraint will remain valid. SELECT conname, convalidated, conrelid::regclass FROM pg_constraint -WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass::text; +WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid; -- Validate the constraint ALTER TABLE fk_partitioned_fk VALIDATE CONSTRAINT fk_partitioned_fk_a_b_fkey; -- All constraints are now valid. SELECT conname, convalidated, conrelid::regclass FROM pg_constraint -WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass::text; +WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid; -- Attaching a child with a NOT VALID constraint. CREATE TABLE fk_partitioned_fk_2 (a int, b int); @@ -1472,7 +1472,7 @@ ALTER TABLE fk_partitioned_fk ATTACH PARTITION fk_partitioned_fk_2 FOR VALUES FR -- The child constraint will also be valid. SELECT conname, convalidated FROM pg_constraint -WHERE conrelid = 'fk_partitioned_fk_2'::regclass ORDER BY oid::regclass::text; +WHERE conrelid = 'fk_partitioned_fk_2'::regclass ORDER BY oid; -- Test case where the child constraint is invalid, the grandchild constraint -- is valid, and the validation for the grandchild should be skipped when a @@ -1486,7 +1486,7 @@ ALTER TABLE fk_partitioned_fk ATTACH PARTITION fk_partitioned_fk_3 FOR VALUES FR -- All constraints are now valid. SELECT conname, convalidated, conrelid::regclass FROM pg_constraint -WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid::regclass::text; +WHERE conrelid::regclass::text like 'fk_partitioned_fk%' ORDER BY oid; DROP TABLE fk_partitioned_fk, fk_notpartitioned_pk; @@ -1505,14 +1505,14 @@ ALTER TABLE fk_notpartitioned_fk ADD CONSTRAINT fk_notpartitioned_fk_a_b_fkey2 -- All constraints will be invalid, and _fkey2 constraints will not be enforced. SELECT conname, conenforced, convalidated FROM pg_constraint -WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid::regclass::text; +WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid; ALTER TABLE fk_notpartitioned_fk VALIDATE CONSTRAINT fk_notpartitioned_fk_a_b_fkey; ALTER TABLE fk_notpartitioned_fk ALTER CONSTRAINT fk_notpartitioned_fk_a_b_fkey2 ENFORCED; -- All constraints are now valid and enforced. SELECT conname, conenforced, convalidated FROM pg_constraint -WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid::regclass::text; +WHERE conrelid = 'fk_notpartitioned_fk'::regclass ORDER BY oid; -- test a self-referential FK ALTER TABLE fk_partitioned_pk ADD CONSTRAINT selffk FOREIGN KEY (a, b) REFERENCES fk_partitioned_pk NOT VALID; @@ -1521,12 +1521,12 @@ CREATE TABLE fk_partitioned_pk_3 PARTITION OF fk_partitioned_pk FOR VALUES FROM CREATE TABLE fk_partitioned_pk_3_1 PARTITION OF fk_partitioned_pk_3 FOR VALUES FROM (2000) TO (2100); SELECT conname, conenforced, convalidated FROM pg_constraint WHERE conrelid = 'fk_partitioned_pk'::regclass AND contype = 'f' -ORDER BY oid::regclass::text; +ORDER BY oid; ALTER TABLE fk_partitioned_pk_2 VALIDATE CONSTRAINT selffk; ALTER TABLE fk_partitioned_pk VALIDATE CONSTRAINT selffk; SELECT conname, conenforced, convalidated FROM pg_constraint WHERE conrelid = 'fk_partitioned_pk'::regclass AND contype = 'f' -ORDER BY oid::regclass::text; +ORDER BY oid; DROP TABLE fk_notpartitioned_fk, fk_partitioned_pk; -- 2.37.1 (Apple Git-137.1)