On 01-01-2021 19:55, Zhihong Yu wrote:
> Hi, Luc:
> Happy New Year.
>
> Looking at BufferAllocExtend()
> in v1-0002-WIP-buffer-alloc-specialized-for-relation-extensi.patch. it
> seems there is duplicate code with the existing BufferAlloc().
>
> It would be good if some refactoring is done by extracting common code
> into a helper function.
>
> Thanks
>
Hi,
Thanks! Happy new year to you too!
Thanks for your suggestion. I would wait a bit and first get some
feedback on the design/approach of my patches before doing the
refactoring. The current code is very much a WIP where I just copied
functions to be able to make specialized variants of them aimed at bulk
extension and then benchmark those. If the refactoring needs to be done
before I can get feedback on the design / approach then let me know.
Kind regards,
Luc
