Re: [patch] Fix checksum verification in base backups for zero page headers

Hi,

Am Dienstag, den 22.09.2020, 16:26 +0200 schrieb Michael Banck:
> Am Mittwoch, den 02.09.2020, 16:50 +0300 schrieb Anastasia Lubennikova:
> > I've looked through the previous discussion. As far as I got it, most of
> > the controversy was about online checksums improvements.
> >
> > The warning about pd_upper inconsistency that you've added is a good
> > addition. The patch is a bit messy, though, because a huge code block
> > was shifted.
> >
> > Will it be different, if you just leave
> > "if (!PageIsNew(page) && PageGetLSN(page) < startptr)"
> > block as it was, and add
> > "else if (PageIsNew(page) && !PageIsZero(page))" ?
>
> Thanks, that does indeed look better as a patch and I think it's fine
> as-is for the code as well, I've attached a v2.

Sorry, forgot to add you as reviewer in the proposed commit message,
I've fixed that up now in V3.

Michael

--
Michael Banck
Projektleiter / Senior Berater
Tel.: +49 2166 9901-171
Fax: +49 2166 9901-100
Email: michael(dot)banck(at)credativ(dot)de

credativ GmbH, HRB Mönchengladbach 12080
USt-ID-Nummer: DE204566209
Trompeterallee 108, 41189 Mönchengladbach
Geschäftsführung: Dr. Michael Meskes, Jörg Folz, Sascha Heuer

Unser Umgang mit personenbezogenen Daten unterliegt
folgenden Bestimmungen: https://www.credativ.de/datenschutz