Remove, from stable branches, the new assertion of no pg_dump OID sort.

nmisch · nmisch · commit 0d2734eac345 · 2025-08-10T13:05:13.000-07:00
Commit 0decd5e recently added the assertion to confirm dump order remains independent of OID values. The assertion remained reachable via DO_DEFAULT_ACL. Given the release wrap tomorrow, make the assertion master-only. Reported-by: Alexander Lakhin <exclusion@gmail.com> Reviewed-by: Robert Haas <robertmhaas@gmail.com> Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://postgr.es/m/d32aaa8d-df7c-4f94-bcb3-4c85f02bea21@gmail.com Backpatch-through: 13-18
diff --git a/src/bin/pg_dump/pg_dump_sort.c b/src/bin/pg_dump/pg_dump_sort.c
@@ -451,7 +451,6 @@ DOTypeNameCompare(const void *p1, const void *p2)
 	 * consequence of the test using "pg_restore -j", which doesn't fully
 	 * constrain OID assignment order.
 	 */
-	Assert(false);
 	return oidcmp(obj1->catId.oid, obj2->catId.oid);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -451,7 +451,6 @@ DOTypeNameCompare(const void p1, const void p2)`
`451`	`451`	`* consequence of the test using "pg_restore -j", which doesn't fully`
`452`	`452`	`* constrain OID assignment order.`
`453`	`453`	`*/`
`454`		`- Assert(false);`
`455`	`454`	`return oidcmp(obj1->catId.oid, obj2->catId.oid);`
`456`	`455`	`}`
`457`	`456`