Robert Haas <robertmhaas@gmail.com> writes:
> On Wed, Feb 20, 2019 at 3:57 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Looking at the patch itself, I agree that a bit more attention to comments
>> is needed, and I wonder whether David has found all the places where
>> it's now necessary to s/NoLock/RowExclusiveLock/. I don't have any
>> other objections.
> I spent some time thinking about that exact issue this morning and
> studying the code to try to figure that out. I wasn't able to find
> any other places that seemed to need updating, but it could be that I
> missed something that David also missed.
Actually, in the wake of b04aeb0a0, we probably need not be too stressed
about the possibility that we missed something. Those assertions wouldn't
detect doing an unwanted lock upgrade, but I think that's unlikely to be
an issue here (or if it is, we have the problem anyway).
regards, tom lane
