Hi,

On Mon, 17 Aug 2026 at 16:11, Zsolt Parragi <[email protected]> wrote:
>
> This issue was reported by Yilin Zhang in another backup related thread[1]
>
> When basebackup finds exactly one corrupted page in a file, the backup
> fails as expected with a per-page warning and then an error. However,
> it doesn't update the statistics, because that update is currently
> inside the per-file summary for 2 or more checksum failures.
>
> This issue seems to exists since the introduction of the feature 
> (6b9e875f728).
>
> The attached patch moves the statistics report into its own block.

I was reading the related thread [1] and I think you are right, it is
an oversight. Your analysis and fix look correct to me.

[1] 
https://postgr.es/m/flat/20180228180856.GE13784%40nighthawk.caipicrew.dd-dns.de

--
Regards,
Nazir Bilal Yavuz
Microsoft


Reply via email to