|From:||Andrey Borodin <x4mmm(at)yandex-team(dot)ru>|
|To:||Ivan Kartyshov <i(dot)kartyshov(at)postgrespro(dot)ru>|
|Cc:||Robert Haas <robertmhaas(at)gmail(dot)com>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Andres Freund <andres(at)anarazel(dot)de>|
|Subject:||Re: [Patch] Checksums for SLRU files|
|Views:||Raw Message | Whole Thread | Download mbox|
> 5 марта 2018 г., в 20:58, Andrey Borodin <x4mmm(at)yandex-team(dot)ru> написал(а):
> I've found that there are few more places with SLRU items per page
I was looking into this patch mainly because I was reviewing other checksums patch in different thread. But the purpose of this patch seems viable for me.
After looking into the patch I'd like to propose some editorialization:
0. Removed GUC: ignore_checksum_failure looks good enough
1. Removed copy on read from disk. Also I do not see a reason to copy page before write
2. multis are upgraded by WAL reset, CLOG is actually upgraded
3. Updated all places where SLRU block size was used
Best regards, Andrey Borodin.
|Next Message||David Rowley||2018-03-19 10:35:43||Re: [HACKERS] path toward faster partition pruning|
|Previous Message||Amit Langote||2018-03-19 10:03:35||Re: [HACKERS] path toward faster partition pruning|