On Tue, Jul 14, 2020 at 11:19 AM Amit Langote <amitlangot...@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. Rest of the changes looked fine to me. Also I noticed that commit message was missing in the patch. Regards, Vignesh EnterpriseDB: http://www.enterprisedb.com