On 2021/09/11 12:21, Fujii Masao wrote:
>
>
> On 2021/07/23 20:07, Ranier Vilela wrote:
>> Em sex., 23 de jul. de 2021 às 07:02, Aleksander Alekseev <aleksander@timescale.com
<mailto:aleksander@timescale.com>>escreveu:
>>
>> Hi hackers,
>>
>> The following review has been posted through the commitfest application:
>> make installcheck-world: tested, passed
>> Implements feature: tested, passed
>> Spec compliant: tested, passed
>> Documentation: tested, passed
>>
>> The patch was tested on MacOS against master `80ba4bb3`.
>>
>> The new status of this patch is: Ready for Committer
>>
>>
>> The second patch seems fine too. I'm attaching both patches to trigger cfbot and to double-check them.
>>
>> Thanks Aleksander, for reviewing this.
>
> I looked at these patches because they are marked as ready for committer.
> They don't change any actual behavior, but look valid to me in term of coding.
> Barring any objection, I will commit them.
> No need to backpatch, why this patch is classified as
> refactoring only.
I found this in the commit log in the patch. I agree that these patches
are refactoring ones. But I'm thinking that it's worth doing back-patch,
to make future back-patching easy. Thought?
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION