Re: Add recovery to pg_control and remove backup_label - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Add recovery to pg_control and remove backup_label
Date
Msg-id ZVvxKDVUH-kvW_qD@paquier.xyz
Whole thread Raw
In response to Re: Add recovery to pg_control and remove backup_label  (Andres Freund <andres@anarazel.de>)
Responses Re: Add recovery to pg_control and remove backup_label
List pgsql-hackers
On Mon, Nov 20, 2023 at 12:37:46PM -0800, Andres Freund wrote:
> Given that, I wonder if what we should do is to just add a new field to
> pg_control that says "error out if backup_label does not exist", that we set
> when creating a streaming base backup

That would mean that one still needs to take an extra step to update a
control file with this byte set, which is something you had a concern
with in terms of compatibility when it comes to external backup
solutions because more steps are necessary to take a backup, no?  I
don't quite see why it is different than what's proposed on this
thread, except that you don't need to write one file to the data
folder to store the backup label fields, but two, meaning that there's
a risk for more mistakes because a clean backup process would require
more steps.

With the current position of the fields in ControlFileData, there are
three free bytes after backupEndRequired, so it is possible to add
that for free.  Now, would you actually need an extra field knowing
that backupStartPoint is around?
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: Why is hot_standby_feedback off by default?
Next
From: Andres Freund
Date:
Subject: Re: Add recovery to pg_control and remove backup_label