On Wed, Oct 1, 2014 at 12:59 PM, Peter Geoghegan <p...@heroku.com> wrote: > On Wed, Oct 1, 2014 at 12:54 PM, Heikki Linnakangas > <hlinnakan...@vmware.com> wrote: >> XactLockTableWait() waits until the end of transaction, that's not you want >> here. If the backend that inserted the promise tuple decides to not proceed >> with the insertion, and removes the promise tuple, the backend waiting on it >> needs to be woken up more or less immediately, not when the transaction >> completes. > > Simon has not been inconsistent here: he has said that deadlocks may > be possible. I happen to think that allowing them would be a major > mistake on our part, but that's another story.
Don't forget that not waiting on XactLockTableWait(), but rather waiting on a "speculative insertion token" wasn't just the thing that made your prototype not deadlock - it was also the thing that made its performance more or less comparable to that of my original value locking design. Your prototype performed only a fraction as well as my design before that last revision. So that's two *excellent* reasons to not use XactLockTableWait() here. -- Peter Geoghegan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers