On Sun, Dec 18, 2022 at 08:56:48PM -0500, Tom Lane wrote: > "Anton A. Melnikov" <aamelni...@inbox.ru> writes: >> 2) In 60684dd83 and b5d63824 there are two changes in the set of specific >> privileges. >> The thing is that in the privileges.sql test there is REVOKE DELETE command >> which becomes pair of REVOKE ALL and GRANT all specific privileges except >> DELETE >> in the result dump. Therefore, any change in the set of specific privileges >> will lead to >> a non-zero dumps diff. >> To avoid this, i propose to replace any specific GRANT and REVOKE in the >> result dumps with ALL. >> This also made in the patch attached. > > Isn't that likely to mask actual bugs?
+ # Replace specific privilegies with ALL + $dump_contents =~ s/^(GRANT\s|REVOKE\s)(\S*)\s/$1ALL /mgx; Yes, this would silence some diffs in the dumps taken from the old and the new clusters. It seems to me that it is one of the things where the original dumps have better be tweaked, as this does not cause a hard failure when running pg_upgrade. While thinking about that, an extra idea popped in my mind as it may be interesting to be able to filter out some of the diffs in some contexts. So what about adding in 002_pg_upgrade.pl a small-ish hook in the shape of a new environment variable pointing to a file adds some custom filtering rules? -- Michael
signature.asc
Description: PGP signature