On Mon, Jun 8, 2015 at 11:52 AM, Michael Paquier <michael.paqu...@gmail.com> wrote: > On Fri, Jun 5, 2015 at 10:45 PM, Fujii Masao wrote: >> Why don't we call InstallXLogFileSegment() at the end of XLogFileCopy()? >> If we do that, the risk of memory leak you're worried will disappear at all. > > Yes, that looks fine, XLogFileCopy() would copy to a temporary file, > then install it definitely. Updated patch attached.
Thanks for updating the patch! Looks good to me. Applied. Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers