Re: pg_subtrans keeps bloating up in the standby
Simon Riggs <simon@2ndquadrant.com>
From: Simon Riggs <simon@2ndQuadrant.com>
To: Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>
Cc: Fujii Masao <masao.fujii@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL-development <pgsql-hackers@postgresql.org>, "Kolb, Harald (NSN - DE/Munich)" <harald.kolb@nsn.com>
Date: 2010-08-30T09:18:49Z
Lists: pgsql-hackers
Attachments
- kagetoldestxmin.patch (text/x-patch) patch
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. This is necessary since CreateCheckpoint is called during end of recovery, though at that point there are still xids in KnownAssignedXids since they aren't removed until slightly later. Not hugely important. Also allows GetOldestXmin to be safely called elsewhere, such as Fujii's earlier patch on this thread. Any objections to commit to both head and 9.0? Will then commit Fujii's patch. -- Simon Riggs www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Training and Services