Hi, On 2019-05-03 09:37:07 +0200, Peter Eisentraut wrote: > REINDEX CONCURRENTLY is still deadlock prone because of > WaitForOlderSnapshots(), so this doesn't actually fix your test case, > but that seems unrelated to this particular issue.
Right. I've not tested the change, but it looks reasonable to me. The change of moving the logic the reset of *heapOid to the unlock perhaps is debatable, but I think it's OK. Greetings, Andres Freund
