Hi Vignesh,
On Tue, Jul 14, 2020 at 10:23 PM vignesh C <vignesh21@gmail.com> wrote:
> On Tue, Jul 14, 2020 at 11:19 AM Amit Langote <amitlangote09@gmail.com> wrote:
> > Sounds fine to me. Although CopyLoadRawBuf() does not seem to a
> > candidate for rigorous code optimization as it does not get called
> > that often.
>
> I thought we could include that change as we are making changes around
> that code.
Sure, done.
> Rest of the changes looked fine to me. Also I noticed that
> commit message was missing in the patch.
Please see the attached v7.
--
Amit Langote
EnterpriseDB: http://www.enterprisedb.com