| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Stephen Frost <sfrost(at)snowman(dot)net> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Magnus Hagander <magnus(at)hagander(dot)net>, Michael Banck <michael(dot)banck(at)credativ(dot)de>, Postgres hackers <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Subject: | Re: pgsql: Add TAP tests for pg_verify_checksums |
| Date: | 2018-10-20 04:58:23 |
| Message-ID: | 20181020045823.GA19190@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
On Sat, Oct 20, 2018 at 12:41:04AM -0400, Stephen Frost wrote:
> I’d also like to give David Steele a chance to comment on the specific API,
> and any other backup tools authors, which I don’t think we should be
> rushing into anyway and I would think we’d only put into master..
By the way, we need to do something for the checksum verification code
in base backups for v11 as well. If you enable checksums and take a
base backup of a build with EXEC_BACKEND, then this creates spurious
checksums failures. That's a bug. So while I agree that having a
larger robust API is fine for HEAD, I would most likely not back-patch
it. This is why I would suggest as a first step for HEAD and v11 to use
a whitelist for base backups, to check for temporary tablespaces in
pg_verify_checksums, to move isRelFileName into src/common/ and to keep
the change minimalistic.
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Stephen Frost | 2018-10-20 05:07:43 | Re: pgsql: Add TAP tests for pg_verify_checksums |
| Previous Message | Stephen Frost | 2018-10-20 04:57:23 | Re: pgsql: Add TAP tests for pg_verify_checksums |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Stephen Frost | 2018-10-20 05:07:43 | Re: pgsql: Add TAP tests for pg_verify_checksums |
| Previous Message | Stephen Frost | 2018-10-20 04:57:23 | Re: pgsql: Add TAP tests for pg_verify_checksums |