On Mon, Aug 30, 2010 at 6:18 PM, Simon Riggs <si...@2ndquadrant.com> wrote: > On Mon, 2010-08-30 at 09:59 +0300, Heikki Linnakangas wrote: > >> However, is it safe to use GetOldestXMin() during recovery? Or to put it >> other way, is GetOldestXMin() functioning correctly during hot standby? >> It only scans through the ProcArray, but not the known-assigned xids >> array. That seems like an oversight that needs to be fixed. > > Patch to implement that attached.
The type of the value which KnownAssignedXidsGetOldestXmin returns should be TransactionId rather than int? Does caller of KnownAssignedXidsGetOldestXmin need to hold ProcArrayLock in (at least) shared mode? If yes, it's helpful to add that note as a comment. > Will then commit Fujii's patch. Thanks. Regards, -- Fujii Masao NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
