Re: Add system identifier to backup manifest - Mailing list pgsql-hackers

From Robert Haas
Subject Re: Add system identifier to backup manifest
Date
Msg-id CA+TgmoYrs8A2j-4KhkjtKp6MwJRwOkXHdXVg4DE6nDOMgDBxMA@mail.gmail.com
Whole thread Raw
In response to Re: Add system identifier to backup manifest  (Amul Sul <sulamul@gmail.com>)
Responses Re: Add system identifier to backup manifest
List pgsql-hackers
On Thu, Feb 15, 2024 at 3:05 PM Amul Sul <sulamul@gmail.com> wrote:
> Kindly have a look at the attached version.

IMHO, 0001 looks fine, except probably the comment could be phrased a
bit more nicely. That can be left for whoever commits this to
wordsmith. Michael, what are your plans?

0002 seems like a reasonable approach, but there's a hunk in the wrong
patch: 0004 modifies pg_combinebackup's check_control_files to use
get_dir_controlfile() rather than git_controlfile(), but it looks to
me like that should be part of 0002.

--
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: make add_paths_to_append_rel aware of startup cost
Next
From: Bertrand Drouvot
Date:
Subject: Re: Synchronizing slots from primary to standby