On Tue, Oct 29, 2019 at 9:19 AM Dilip Kumar <dilipbalaut(at)gmail(dot)com> wrote:
>
> Few comments:
> 1.
> * The act of allocating pages to recycle may have invalidated the
> - * results of our previous btree reserch, so repeat it. (We could
> + * results of our previous btree search, so repeat it. (We could
> * recheck whether any of our split-avoidance strategies that were
>
Fixed
> I think the old comment meant "btree research" but you changed to "btree search"
>
> 2.
> /* copy&pasted from .../src/backend/utils/adt/datetime.c
> - * and changesd struct pg_tm to struct tm
> + * and changes struct pg_tm to struct tm
> */
> Seems like this comment meant "Changed struct pg_tm to struct tm"
Fixed
Thanks for the review.
I have attached the updated patch with the fixes.
Regards,
Vignesh
EnterpriseDB: http://www.enterprisedb.com