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.banck@credativ.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
