| 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 |
| 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() |