On Thu, 2010-01-28 at 10:43 -0500, Tom Lane wrote:
> sriggs@postgresql.org (Simon Riggs) writes:
> > Log Message:
> > -----------
> > Use malloc() in GetLockConflicts() when called InHotStandby to avoid repeated
> > palloc calls. Current code assumed this was already true, so this is a bug fix.
>
> It seems pretty ugly to allocate the same pointer sometimes with malloc
> and sometimes with palloc. How about palloc'ing it in TopMemoryContext
> when you want it to be persistent?
Yes, that sounds more graceful. Will do.
--
Simon Riggs www.2ndQuadrant.com