Hi, On 31/12/13 13:56, Peter Geoghegan wrote: > I think that this is a worthwhile effort, but like Tom I don't think > that it's universally true that these waiters are waiting on a tuple > lock. Most obviously, the XactLockTableWait() call you've modified > within nbtinsert.c is not.
This is why I don't set the tuple data in this case:
XactLockTableWaitSetupErrorContextCallback(rel, NULL);
The second argument is the tuple argument. If it is set to NULL in the
error context callback, all output regarding tuple are suppressed.
> ISTM that you should be printing just the value and the unique index
> there, and not any information about the tuple proper.
Good point, I will have a look at this.
> For better direction about where that new
> infrastructure ought to live, you might find some useful insight from
> commit 991f3e5ab3f8196d18d5b313c81a5f744f3baaea.
Thanks for the pointer!
But, to be honest, I am still unsure where to put this. As far as I
understand this commit has substantial parts in relcache.c and
elog.c – both don't seem to be very good fitting places?
Regards,
--
Christian Kruse http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
pgpFTITwzCXuK.pgp
Description: PGP signature
