On 11 August 2015 at 14:53, Amit Kapila <amit.kapil...@gmail.com> wrote:
> One more point here why do we need CommitLock before calling > SimpleLruReadPage_ReadOnly() in the patch and if it is not required, > then can we use LWLockAcquire(shared->buffer_locks[slotno], LW_EXCLUSIVE); > instead of CommitLock? > That prevents read only access, not just commits, so that isn't a better suggestion. -- Simon Riggs http://www.2ndQuadrant.com/ <http://www.2ndquadrant.com/> PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services