| From: | Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com> |
|---|---|
| To: | Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> |
| Cc: | Daniel Gustafsson <daniel(at)yesql(dot)se>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Offline data checksum changes can cause incorrect checksum state on standbys |
| Date: | 2026-08-31 11:38:40 |
| Message-ID: | apVnwOuJBb4rHk+B@bdtpg |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On Mon, Aug 31, 2026 at 11:55:01AM +0100, Zsolt Parragi wrote:
> > Yeah, I see your point that the underlying issue is the non WAL logged change.
> > My concern was not about a pg_control change in itself, but that addressing this
> > case might require further changes to the current pg_control design, which adds
> > risk this close to RC1.
>
> I don't think we can get this completely right without both wal and
> pg_control changes (and possibly some extra connection-time
> communication between the standby and primary to provide early
> reporting instead of delayed in the most common cases)
> If we want to move toward the error direction in pg20 (which we
> should) we will have to log that an offline change happened.
>
> I attached v6 which for now documents the current behavior as-is, and
> other than that I reorganized the test files.
Thanks! I did not look in details but it looks like that pg_upgrade --check against
a running source cluster with checksums enabled will still fail (finding === 2 in
[0], tested with the test shared in [1]).
[0]: https://postgr.es/m/apUL3N4IE934qJ08%40bdtpg
[1]: https://postgr.es/m/apU4/hmRv/4gv20W%40bdtpg
Regards,
--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Akshay Joshi | 2026-08-31 11:49:33 | Re: [PATCH] Add pg_get_table_ddl() to reconstruct CREATE TABLE statements |
| Previous Message | Pavlo Golub | 2026-08-31 11:35:28 | Re: [PATCH v4] Add pg_current_vxact_id() function |