Re: Incorrect snapshots while promoting hot standby node when 2PC is used
x4mmm@yandex-team.ru
From: Andrey Borodin <x4mmm@yandex-team.ru>
To: Andres Freund <andres@anarazel.de>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>,
Michael Paquier <michael.paquier@gmail.com>,
Simon Riggs <simon@2ndquadrant.com>,
Thomas Munro <thomas.munro@gmail.com>
Date: 2021-05-04T06:58:36Z
Lists: pgsql-hackers
> 3 мая 2021 г., в 23:10, Andres Freund <andres@anarazel.de> написал(а): > > Hi, > > On 2021-05-01 17:35:09 +0500, Andrey Borodin wrote: >> I'm investigating somewhat resemblant case. >> We have an OLTP sharded installation where shards are almost always under rebalancing. Data movement is implemented with 2pc. >> Switchover happens quite often due to datacenter drills. The installation is running on PostgreSQL 12.6. > > If you still have the data it would be useful if you could check if the > LSNs of the corrupted pages are LSNs from shortly after standby > promotion/switchover? That's a neat idea, I'll check if I can restore backup with corruptions. I have a test cluster with corruptions, but it has undergone tens of switchovers... >> Or, perhaps, it looks more like a hardware problem? Data_checksums are >> on, but few years ago we observed ssd firmware that was loosing >> updates, but passing checksums. I'm sure that we would benefit from >> having separate relation fork for checksums or LSNs. > > Right - checksums are "page local". They can only detect if a page is > corrupted, not if e.g. an older version of the page (with correct > checksum) has been restored. While there are schemes to have stronger > error detection properties, they do come with substantial overhead (at > least the ones I can think of right now). We can have PTRACK-like fork with page LSNs. It can be flushed on checkpoint and restored from WAL on crash. So we always can detect stale page version. Like LSN-track :) We will have much faster rewind and delta-backups for free. Though I don't think it worth an effort until we at least checksum CLOG. Thanks! Best regards, Andrey Borodin.
Commits
-
Fix snapshot builds during promotion of hot standby node with 2PC
- e2b2a9e1c3d5 9.6.24 landed
- 8a6a1fe07e06 10.19 landed
- 0a561d4d0016 11.14 landed
- 3c3f118d503c 12.9 landed
- 194e535a07b3 13.5 landed
- 828f7f0009ed 14.1 landed
- 8a4237908c0f 15.0 landed