Re: Allow pg_archivecleanup to remove backup history files
torikoshia <torikoshia@oss.nttdata.com>
From: torikoshia <torikoshia@oss.nttdata.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Fujii Masao <masao.fujii@oss.nttdata.com>, Kyotaro Horiguchi
<horikyota.ntt@gmail.com>, Bharath Rupireddy
<bharath.rupireddyforpostgres@gmail.com>, pgsql-hackers@postgresql.org
Date: 2023-06-13T15:49:39Z
Lists: pgsql-hackers
Attachments
- v6-0001-Introduce-pg_archivecleanup-into-getopt_long.patch (text/x-diff) patch v6-0001
- v6-0002-Allow-pg_archivecleanup-to-remove-backup-history-.patch (text/x-diff) patch v6-0002
- v6-0003-Refactored-to-reduce-the-nesting-level.patch (text/x-diff) patch v6-0003
On 2023-06-12 16:33, Michael Paquier wrote:
> On Fri, Jun 09, 2023 at 12:32:15AM +0900, Fujii Masao wrote:
Thanks for reviewing!
>> printf(_(" -n, --dry-run dry run, show the names of
>> the files that would be removed\n"));
>> + printf(_(" -b, --clean-backup-history clean up files including
>> backup history files\n"));
>>
>> Shouldn't -b option be placed in alphabetical order?
>
> +1.
Modified the place.
--
Regards,
--
Atsushi Torikoshi
NTT DATA CORPORATION
Commits
-
pg_archivecleanup: Add --clean-backup-history
- 3f8c98d0b631 17.0 landed
-
pg_archivecleanup: Refactor loop doing old segment removals
- 4a7556f77c44 17.0 landed
-
Introduce long options in pg_archivecleanup
- dd7c60f1935a 17.0 landed