Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1vx5xF-00F9Kw-24 for pgsql-bugs@arkaria.postgresql.org; Mon, 02 Mar 2026 16:17:09 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.96) (envelope-from ) id 1vx5xE-001mM2-0M for pgsql-bugs@arkaria.postgresql.org; Mon, 02 Mar 2026 16:17:08 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1vx5xD-001mLt-2l for pgsql-bugs@lists.postgresql.org; Mon, 02 Mar 2026 16:17:08 +0000 Received: from sss.pgh.pa.us ([68.162.161.243]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1vx5xC-000000002cN-2Qeh for pgsql-bugs@lists.postgresql.org; Mon, 02 Mar 2026 16:17:07 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 622GH337846353; Mon, 2 Mar 2026 11:17:03 -0500 From: Tom Lane To: Robert Haas cc: =?UTF-8?Q?=C3=81lvaro_Herrera?= , Virender Singla , pgsql-bugs@lists.postgresql.org, Aniket Jha Subject: Re: Major Version Upgrade failure due to orphan roles entries in catalog In-reply-to: References: <202502131716.7mgkcnrem2hn@alvherre.pgsql> <2939991.1740089974@sss.pgh.pa.us> <179448.1772033773@sss.pgh.pa.us> <265501.1772038216@sss.pgh.pa.us> <296083.1772041154@sss.pgh.pa.us> <301798.1772044229@sss.pgh.pa.us> <304751.1772045891@sss.pgh.pa.us> <1103272.1772227345@sss.pgh.pa.us> <1104379.1772227731@sss.pgh.pa.us> <1106526.1772229127@sss.pgh.pa.us> Comments: In-reply-to Robert Haas message dated "Mon, 02 Mar 2026 09:19:29 -0500" MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-ID: <846351.1772468223.1@sss.pgh.pa.us> Content-Transfer-Encoding: 8bit Date: Mon, 02 Mar 2026 11:17:03 -0500 Message-ID: <846352.1772468223@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Robert Haas writes: > On Fri, Feb 27, 2026 at 4:52 PM Tom Lane wrote: >> Fair ... do you have a suggestion for less confusing names? >> I considered naming the new variable "dump_this_grantor", but thought >> it was longer without being more helpful ... but maybe you disagree. > Personally, I'd find the extra verbosity helpful, but I don't care > that much if you see it otherwise. If you think that's better, I'm happy to make it so. An author is seldom the best judge of readability of his code. Pushed like that. regards, tom lane