On Wed, Jan 19, 2022 at 7:16 AM Dipesh Pandit <dipesh.pan...@gmail.com> wrote:
> I have done initial testing and
> working on updating the test coverage.

I spent some time thinking about test coverage for the server-side
backup code today and came up with the attached (v12-0003). It does an
end-to-end test that exercises server-side backup and server-side
compression and then untars the backup and validity-checks it using
pg_verifybackup. In addition to being good test coverage for these
patches, it also plugs a gap in the test coverage of pg_verifybackup,
which currently has no test case that untars a tar-format backup and
then verifies the result. I couldn't figure out a way to do that back
at the time I was working on pg_verifybackup, because I didn't think
we had any existing precedent for using 'tar' from a TAP test. But it
was pointed out to me that we do, so I used that as the model for this
test. It should be easy to generalize this test case to test lz4 and
zstd as well, I think. But I guess we'll still need something
different to test what your patch is doing.

-- 
Robert Haas
EDB: http://www.enterprisedb.com

Attachment: v12-0002-Server-side-gzip-compression.patch
Description: Binary data

Attachment: v12-0003-Test-server-side-backup-backup-compression-and-p.patch
Description: Binary data

Attachment: v12-0001-Support-base-backup-targets.patch
Description: Binary data

Reply via email to