Jeff Davis <pg...@j-davis.com> writes: > On Wed, 2011-06-08 at 17:29 -0500, Kevin Grittner wrote: >> Heikki Linnakangas <heikki.linnakan...@enterprisedb.com> wrote: >>> AFAICS, the check for page lock is actually unnecessary.
>> Absolutely correct. Patch attached. > I like the change, but the comment is slightly confusing. I've committed this patch with comment rewording along the lines suggested by Jeff. I also moved the CheckForSerializableConflictIn call to just before, instead of just after, the RelationGetBufferForTuple call. We no longer have to do it after, since we don't need to know which buffer to pass, and it should buy some more low-level parallelism to run the SSI checks while not holding exclusive lock on the eventual target buffer. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers