NotificationsYou must be signed in to change notification settings
Fork15
Star24

Fix diff in the test moc100 on PG16#57

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

vitcpp merged 1 commit intopostgrespro:masterfromvitcpp:fix-test-moc100-pg16

Aug 26, 2023

Merged

Fix diff in the test moc100 on PG16#57

vitcpp merged 1 commit intopostgrespro:masterfromvitcpp:fix-test-moc100-pg16

Aug 26, 2023

Conversation

Copy link

Contributor

vitcpp commentedAug 25, 2023

No description provided.

Fix diff in the test moc100 on PG16

2116310

Copy link

Contributor

esabol commentedAug 25, 2023

I confused as to why this change is working on PG 10-15. I thought only the PG16 build broke and the test output was unaltered on PG 10-15.

Copy link

ContributorAuthor

vitcpp commentedAug 25, 2023

@esabol expected/moc100_4.out is applied only in case of PG16:

https://www.postgresql.org/docs/current/regress-variant.html

Other "golden" files expected/moc100_*.out are applied for other versions of postgres. You can see that the beginning of each file contains some output that is specific for each version. It plays the role of a filter. For unsuitable versions, it will produce differences and the golden file will be skipped.