Re: Major Version Upgrade failure due to orphan roles entries in catalog - Mailing list pgsql-bugs

Laurenz Albe <laurenz.albe@cybertec.at> writes:
> Thanks for the explanation.  That might be worth a comment.

The adjacent comment already says

    /*
     * Advance command counter so we can see new record; else tests in
     * AddRoleMems may fail.
     */

so I didn't see anything to add there.  Maybe "We can skip this in
cases where we will not call AddRoleMems"?  Or maybe the better answer
is to conclude that the whole idea of not calling
CommandCounterIncrement unconditionally is too fragile and not worth
expending brain cells on, and just rip out the if-test.

            regards, tom lane