Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.92) (envelope-from ) id 1jGGD3-0001Zg-ST for pgsql-committers@arkaria.postgresql.org; Mon, 23 Mar 2020 06:05:14 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1jGGD1-0000dV-MV for pgsql-committers@arkaria.postgresql.org; Mon, 23 Mar 2020 06:05:11 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1jGGD1-0000dO-BX for pgsql-committers@lists.postgresql.org; Mon, 23 Mar 2020 06:05:11 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jGGCy-0005we-DR; Mon, 23 Mar 2020 06:05:09 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id 02N657cH023197; Mon, 23 Mar 2020 02:05:07 -0400 From: Tom Lane To: Jeff Davis cc: pgsql-committers@lists.postgresql.org Subject: Re: pgsql: Disk-based Hash Aggregation. In-reply-to: References: Comments: In-reply-to Jeff Davis message dated "Wed, 18 Mar 2020 22:54:49 -0000" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <23195.1584943506.1@sss.pgh.pa.us> Date: Mon, 23 Mar 2020 02:05:06 -0400 Message-ID: <23196.1584943506@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk Jeff Davis writes: > Disk-based Hash Aggregation. I noticed that the regression tests seemed suddenly slower than they have been. A bit of poking around reveals that this patch made groupingsets.sql take approximately 8X longer than it used to, and more than twice as long as any other core regression test. This is absolutely, positively, not acceptable for a test that gets run hundreds of times a day by lots of people and buildfarm animals. If there's no way to test the feature in some significantly-cheaper way, perhaps we should move this test out to a separate script that's not run by default. regards, tom lane