Re: Spurious standby query cancellations

On 14 September 2015 at 12:00, Jeff Janes <jeff.janes@gmail.com> wrote:

 

It's now possible to fix this by putting a lock wait on the actual lock request, which wasn't available when I first wrote that, hence the crappy wait loop. Using the timeout handler would now be the preferred way to solve this. We can backpatch that to 9.3 if needed, where they were introduced.

There's an example of how to use lock waits further down on ResolveRecoveryConflictWithBufferPin(). Could you have a look at doing it that way?

It looks like this will take some major surgery.  The heavy weight lock manager doesn't play well with others when it comes to timeouts other than its own.  LockBufferForCleanup is a simple retry loop, but the lock manager is much more complicated than that.

Not sure I understand this objection. I can't see a reason that my proposal wouldn't work.