On 2023-06-21 11:59, Kyotaro Horiguchi wrote:
> At Tue, 20 Jun 2023 22:27:36 +0900, torikoshia
> <torikoshia@oss.nttdata.com> wrote in
>> On 2023-06-19 14:37, Michael Paquier wrote:
>> > On Mon, Jun 19, 2023 at 11:24:29AM +0900, torikoshia wrote:
>> >> Thanks, now I understand what you meant.
>> > If I may ask, why is the refactoring of 0003 done after the feature in
>> > 0002? Shouldn't the order be reversed? That would make for a cleaner
>> > git history.
>> > --
>> > Michael
>>
>> Agreed.
>> Reversed the order of patches 0002 and 0003.
>
> Yeah, that is a possible division. However, I meant that we have room
> to refactor and decrease the nesting level even further, considering
> that 0003 already does this to some extent, when I suggested it. In
> that sense, moving the nest-reduction part of 0003 into 0002 makes us
> possible to focus on the point of this patch.
Thanks for the comment, it seems better than v9 patch.
> What do you think about the attached version?
--v10-0002-Preliminary-refactoring-for-a-subsequent-patch.patch
+ * Also we skip backup history files when --clean-backup-history
+ * is not specified.
+ */
+ if (!IsXLogFileName(walfile) && !IsPartialXLogFileName(walfile))
+ continue;
I think this comment should be located in 0003.
Attached updated patches.
--
Regards,
--
Atsushi Torikoshi
NTT DATA CORPORATION