Skip to content

Update golden schema comparison in schema test#2805

Merged
weiminyu merged 1 commit into
google:masterfrom
weiminyu:pgdump_format_change
Aug 18, 2025
Merged

Update golden schema comparison in schema test#2805
weiminyu merged 1 commit into
google:masterfrom
weiminyu:pgdump_format_change

Conversation

@weiminyu

@weiminyu weiminyu commented Aug 18, 2025

Copy link
Copy Markdown
Collaborator

Postgresql-17.6 introduces two new lines in pg_dump output as a security feature: \restrict {HASH} and \unrestrict {HASH}.

We filter out lines starting with these two prefixes when comparing schemas.

The db upgrade also adds two empty lines to the pg_dump output. We know ignore
all blank lines (which includes empty lines) when comparing schemas.


This change is Reviewable

Postgresql-17.6 introduces two new lines in pg_dump output as a
security feature: `\restricted {HASH}` and `\unrestricted {HASH}`.

We filter out lines starting with these two prefixes when comparing
schemas.

The db upgrade also adds two empty lines to the pg_dump output. We
know ignore all empty lines when comparing schemas.
@weiminyu
weiminyu requested a review from ptkach August 18, 2025 19:47

@ptkach ptkach left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @weiminyu)

@weiminyu
weiminyu enabled auto-merge August 18, 2025 20:21
@weiminyu
weiminyu added this pull request to the merge queue Aug 18, 2025
Merged via the queue into google:master with commit 1164070 Aug 18, 2025
9 checks passed
@weiminyu
weiminyu deleted the pgdump_format_change branch August 18, 2025 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants