Tom Lane escribió:
> Alvaro Herrera <alvhe...@commandprompt.com> writes:
> > Tom Lane escribi�:
> >> I don't think that testing rowMarks is the right thing at all here.
> >> That tells you whether it's a SELECT FOR UPDATE, but actually we
> >> want any cursor (and only cursors) to have a private snapshot.
> 
> > The attached patch implements this.  I intend to apply to 8.4 and HEAD
> > shortly.
> 
> Looks sane.  Can we add a short regression test sequence that checks
> for this?

Something is wrong with the patch :-(  I'm getting
WARNING:  Snapshot reference leak: Snapshot 0x1be5840 still referenced
with a simple test case.  Still investigating.

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

-- 
Sent via pgsql-general mailing list (pgsql-general@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-general

Reply via email to