Skip to content

Commit e43fae1

Browse files
pg_dump: Fix object-type sort priority for large objects.
Commit a45c78e moved large object metadata from SECTION_PRE_DATA to SECTION_DATA but neglected to move PRIO_LARGE_OBJECT in dbObjectTypePriorities accordingly. While this hasn't produced any known live bugs, it causes problems for a proposed patch that optimizes upgrades with many large objects. Fixing the priority might also make the topological sort step marginally faster by reducing the number of ordering violations that have to be fixed. Reviewed-by: Nitin Motiani <nitinmotiani@google.com> Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://postgr.es/m/aBkQLSkx1zUJ-LwJ%40nathan Discussion: https://postgr.es/m/aG_5DBCjdDX6KAoD%40nathan Backpatch-through: 17
1 parent c1c6169 commit e43fae1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/bin/pg_dump/pg_dump_sort.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,10 @@ enum dbObjectTypePriorities
7777
PRIO_TABLE_ATTACH,
7878
PRIO_DUMMY_TYPE,
7979
PRIO_ATTRDEF,
80-
PRIO_LARGE_OBJECT,
8180
PRIO_PRE_DATA_BOUNDARY, /* boundary! */
8281
PRIO_TABLE_DATA,
8382
PRIO_SEQUENCE_SET,
83+
PRIO_LARGE_OBJECT,
8484
PRIO_LARGE_OBJECT_DATA,
8585
PRIO_POST_DATA_BOUNDARY, /* boundary! */
8686
PRIO_CONSTRAINT,

0 commit comments

Comments
 (0)