Re: More issues with pg_verify_checksums and checksum verification in base backups
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Michael Banck <michael.banck@credativ.de>
Cc: pgsql-hackers@postgresql.org
Date: 2019-08-05T07:30:52Z
Lists: pgsql-hackers
On Sat, Aug 03, 2019 at 06:47:48PM +0200, Michael Banck wrote: > first off, a bit of a meta-question: Did the whitelist approach die > completely, or are we going to tackle it again for v13 or later? At this stage, it is burried. Amen. > This is something I still have in the test suite of my pg_checksums > fork, cause I never reverted that one from isRelFile() back to > skipfile() (so it doesn't fail on the above cause `123.' is not > considered a relation file worth checksumming). We could actually fix this one. It is confusing to have pg_checksums generate a report about a segment number which is actually incorrect. > Independently of the whitelist/blacklist question, I believe > pg_checksums should not error out as soon as it encounters a weird looking > file, but either (i) still checksum it or (ii) skip it? Or is that to be > considered a pilot error and it's fine for pg_checksums to fold? That's actually the distinctions between the black and white lists which would have handled that. -- Michael
Commits
-
Fix various checksum check problems for pg_verify_checksums and base backups
- 19516afdf167 11.2 landed
- 5c995139752f 12.0 landed
-
Switch pg_verify_checksums back to a blacklist
- 85036308dc58 11.2 landed
- a1c91dd1108c 12.0 landed