Re: pg_control_checkpoint(): add "data_checksum_version" (Pg19)?

From: Ian Lawrence Barwick <barwick(at)gmail(dot)com>
To: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Cc: Daniel Gustafsson <daniel(at)yesql(dot)se>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: pg_control_checkpoint(): add "data_checksum_version" (Pg19)?
Date: 2026-08-10 14:21:33
Message-ID: CAB8KJ=g1FxfWRetBOQJCVrZ7mPEO-axFEW2ckkHLYLk-Pqro1w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2026年8月10日(月) 22:30 Fujii Masao <masao(dot)fujii(at)gmail(dot)com>:
>
> On Mon, Aug 10, 2026 at 9:39 PM Ian Lawrence Barwick <barwick(at)gmail(dot)com> wrote:
> > The function "get_checksum_state_string()" which performs the conversion is in
> > src/backend/access/rmgrdesc/xlogdesc.c, which AFAIR can't be linked from
> > frontend utilities, so we'd either have to move that somewhere else (maybe
> > src/common/controldata_utils.c?) or duplicate it.
>
> Agreed. I think moving it to common is better than duplicating the mapping.
>
> src/common/controldata_utils.c seems like a reasonable place, since
> this state is stored in the control/checkpoint data and both pg_controldata
> and pg_control_checkpoint() already use controldata_utils.

TBH it feels a little odd there because it's not really specific to
the controldata
output functions, but it has to go somewhere, I guess.

Updated patch with that and changes per your other suggestions attached.

Regards

Ian Barwick

Attachment Content-Type Size
v3-0001-Add-data_checksum_state-to-pg_control_checkpoint.patch text/x-patch 9.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bertrand Drouvot 2026-08-10 14:45:16 Re: Redesign per-backend statistics
Previous Message Pierre Forstmann 2026-08-10 13:48:02 Re: Avoid recalculating pgprocno in ProcArrayAdd()