Re: [HACKERS] [patch] reorder tablespaces in basebackup tar streamfor backup_label

On Fri, Mar 17, 2017 at 7:18 PM, Michael Banck
<michael.banck@credativ.de> wrote:
> Hi,
>
> Am Freitag, den 17.03.2017, 10:50 +0900 schrieb Michael Paquier:
>> The comment block format is incorrect. I would think as well that this
>> comment should say it is important to have the main tablespace listed
>> last it includes the WAL segments, and those need to contain all the
>> latest WAL segments for a consistent backup.
>
> How about the attached? The comment now reads as follows:
>
> |Add a node for the base directory. If WAL is included, the base
> |directory has to be last as the WAL files get appended to it. If WAL
> |is not included, send the base directory first, so that the
> |backup_label file is the first file to be sent.

Close enough, still not that:
https://www.postgresql.org/docs/current/static/source-format.html

>> FWIW, I have no issue with changing the ordering of backups the way
>> you are proposing here as long as the comment of this code path is
>> clear.
>
> OK, great, let's see what the committers think then.

Still that's a minor point, so I am making that ready for committer.
-- 
Michael