Re: Always have pg_dump write rules in a consistent order

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andreas Karlsson <andreas(at)proxel(dot)se>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Always have pg_dump write rules in a consistent order
Date: 2024-11-04 18:32:53
Message-ID: 2787458.1730745173@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andreas Karlsson <andreas(at)proxel(dot)se> writes:
> When working on a new feature for PostgreSQL I noticed that the
> pg_upgrade tests became flaky due to rules being dumped in a different
> order between the original cluster and the upgraded cluster. (For
> context: my regress scripts left a bunch of views with dependency
> circles which I had forgot to clean up.)

> So I propose that we sort rules on (schema, name, relation_name) instead
> of (schema, name, oid) like it currently does. The patch itself is small
> enough and makes it easier to test that we upgrade rules and views
> correctly with pg_upgrade.

Seems reasonable. Pushed with some trivial cosmetic adjustments
to make it look more like the identical adjacent cases for policies
and triggers.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Masahiko Sawada 2024-11-04 18:34:28 Re: New "raw" COPY format
Previous Message Masahiko Sawada 2024-11-04 18:23:20 Re: Fix a typo in the comment of gistdoinsert()