On Mon, Feb 9, 2015 at 7:58 PM, Fujii Masao <masao.fu...@gmail.com> wrote: > MemoryContextAllocExtended() was added, so isn't it time to replace palloc() > with MemoryContextAllocExtended(CurrentMemoryContext, MCXT_ALLOC_NO_OOM) > in allocate_recordbuf()?
Yeah, let's move on here, but not with this patch I am afraid as this breaks any client utility using xlogreader.c in frontend, like pg_xlogdump or pg_rewind because MemoryContextAllocExtended is not available in frontend, only in backend. We are going to need something like palloc_noerror, defined on both backend (mcxt.c) and frontend (fe_memutils.c) side. Btw, the huge flag should be used as well as palloc only allows allocation up to 1GB, and this is incompatible with ~9.4 where malloc is used. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
