Andres Freund <and...@anarazel.de> writes: > I should have been more precise - what I meant was a timeout.c API that allows > the caller to pass in "now", which in this case we'd get from > GetCurrentTransactionStopTimestamp(), which would avoid the additional > timestamp computation.
I don't care for that one bit: it makes the accuracy of all timeouts dependent on how careful that caller is to provide an up-to-date "now". In the example at hand, there is WAY too much code between SetCurrentTransactionStopTimestamp() and the timer arming to make me think the results will be acceptable. regards, tom lane