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 1tyAb9-003L0f-Ky for pgsql-hackers@arkaria.postgresql.org; Fri, 28 Mar 2025 14:22: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 1tyAb7-00B0zG-Nf for pgsql-hackers@arkaria.postgresql.org; Fri, 28 Mar 2025 14:22:13 +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.94.2) (envelope-from ) id 1tyAb7-00B0w7-Cq for pgsql-hackers@lists.postgresql.org; Fri, 28 Mar 2025 14:22:13 +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.96) (envelope-from ) id 1tyAb5-001dTP-1x for pgsql-hackers@lists.postgresql.org; Fri, 28 Mar 2025 14:22:12 +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 52SEM89O2368896; Fri, 28 Mar 2025 10:22:08 -0400 From: Tom Lane To: Alvaro Herrera cc: Ashutosh Bapat , Michael Paquier , vignesh C , Daniel Gustafsson , Peter Eisentraut , PostgreSQL Hackers Subject: Re: Test to dump and restore objects left behind by regression In-reply-to: <202503281411.io3xbmyibj7t@alvherre.pgsql> References: <202503281411.io3xbmyibj7t@alvherre.pgsql> Comments: In-reply-to Alvaro Herrera message dated "Fri, 28 Mar 2025 15:11:15 +0100" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <2368894.1743171728.1@sss.pgh.pa.us> Date: Fri, 28 Mar 2025 10:22:08 -0400 Message-ID: <2368895.1743171728@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Alvaro Herrera writes: > Hmm, I didn't mean that we'd maintain a separate schedule. I meant that > we'd take the existing schedule, then apply some Perl magic to it that > grep-outs the tests that we know to contribute nothing, and generate a > new schedule file dynamically. We don't need to maintain a separate > schedule file. This seems like a fundamentally broken approach to me. The entire argument for using the core regression tests as a source of data to test dump/restore is that, more or less "for free", we can expect to get coverage when new SQL language features are added. That's always been a little bit questionable --- there's a temptation to drop objects again at the end of a test script. But with this, it becomes a complete crapshoot whether the objects you need will be included in the dump. I think instead of going this direction, we really need to create a separately-purposed script that simply creates "one of everything" without doing anything else (except maybe loading a little data). I believe it'd be a lot easier to remember to add to that when inventing new SQL than to remember to leave something behind from the core regression tests. This would also be far faster to run than any approach that involves picking a random subset of the core test scripts. regards, tom lane