On Thu, 2010-01-28 at 10:43 -0500, Tom Lane wrote:
> [email protected] (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


-- 
Sent via pgsql-committers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

Reply via email to