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.94.2) (envelope-from ) id 1rpVAN-0073Bb-2k for pgsql-hackers@arkaria.postgresql.org; Wed, 27 Mar 2024 15:26:15 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1rpVAM-00GEFH-65 for pgsql-hackers@arkaria.postgresql.org; Wed, 27 Mar 2024 15:26:14 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rpVAL-00GEF9-Td for pgsql-hackers@lists.postgresql.org; Wed, 27 Mar 2024 15:26:13 +0000 Received: from sss.pgh.pa.us ([68.162.161.243]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rpVAG-006oGq-6B for pgsql-hackers@postgresql.org; Wed, 27 Mar 2024 15:26:13 +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 42RFQ3cR3308710; Wed, 27 Mar 2024 11:26:03 -0400 From: Tom Lane To: Daniel Gustafsson cc: Andres Freund , Peter Eisentraut , vignesh C , Alvaro Herrera , Robert Haas , PostgreSQL Hackers Subject: Re: pgsql: Clean up role created in new subscription test. In-reply-to: <6BF8531D-355E-4D41-BA13-653505F97684@yesql.se> References: <202312011219.aikuwve3cbku@alvherre.pgsql> <57373BD5-C0D9-486D-9096-276FE91411D7@yesql.se> <1ECB500B-F76F-4BBE-AE4D-91DDC5E59D7E@yesql.se> <20240325184824.uxbh6o7w4442o4nv@awork3.anarazel.de> <6BF8531D-355E-4D41-BA13-653505F97684@yesql.se> Comments: In-reply-to Daniel Gustafsson message dated "Wed, 27 Mar 2024 14:42:02 +0100" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <3308708.1711553163.1@sss.pgh.pa.us> Date: Wed, 27 Mar 2024 11:26:03 -0400 Message-ID: <3308709.1711553163@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Daniel Gustafsson writes: > The only option is to make the check opt-in via a command-line parameter for > use it in the main regress tests, and not use it for the contrib tests. The > attached v7 does that and passes CI, but I wonder if it's worth it all with > that restriction? Yeah, that seems hardly worth it --- and it's only an accident of implementation that we don't run the main tests in parallel with something else, anyway. Too bad, but ... regards, tom lane