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 1v6lgJ-009O81-Q9 for pgsql-hackers@arkaria.postgresql.org; Thu, 09 Oct 2025 08:07:23 +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 1v6lgH-001dhH-Ii for pgsql-hackers@arkaria.postgresql.org; Thu, 09 Oct 2025 08:07:22 +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 1v6lgH-001dh7-95 for pgsql-hackers@lists.postgresql.org; Thu, 09 Oct 2025 08:07:22 +0000 Received: from mail-yw1-x112a.google.com ([2607:f8b0:4864:20::112a]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.96) (envelope-from ) id 1v6lgF-000onZ-1p for pgsql-hackers@postgresql.org; Thu, 09 Oct 2025 08:07:21 +0000 Received: by mail-yw1-x112a.google.com with SMTP id 00721157ae682-77f9fb2d9c5so7287047b3.0 for ; Thu, 09 Oct 2025 01:07:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1759997240; x=1760602040; darn=postgresql.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=CYbPISX6w4r1MBKiZVSWpZd/AOvWSjlTLrDXWEU1SCA=; b=Maa2nca36cx8G/35tuFuV331hga4Qh2KpbBh/MjgvdzMsWBJeokkkL+7gvaPWdjcjK b+2k5H2dqUPE0dKhCn86vv9VyhcQHDd3YrjEBd6o6q9/tzKKwojlGdaloBWhEYtwzJc5 X8lETcTVPGPuw83FNU39RXFy4iY9k7mvJw/a1zu+7VsoBV5Fn5NklyO+KhgWEGCL3qsh aLdES+utKMZ9h4XhUcMeCEk2nhccc88GBVXDwev7rfsMVF/rvOTUdUQNnHJIF4hUfGXI COK7wj0eHXRu/MuabFTtycyhDPoDpxuzqsUxOvUxQpcZOmGUvc8bd6DHFwhp6WNBLv4U vMEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1759997240; x=1760602040; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=CYbPISX6w4r1MBKiZVSWpZd/AOvWSjlTLrDXWEU1SCA=; b=wJigfPwV+/S2rafNb0I9JhmT2aTMC3W5xlrHT83m3Gx2vSdNyzHndg6MBW8V14U+dc uCv9lWmpGZU5+Anr+nTyhrzCypwR43y7ZCAN2cB+0kYMt9qNx7e2vFv4rqF35n0Hq4jO jW/WldwFAgkFQwmYrtpaBzxupaKMyd6TPg2w9sOkdyWfaPfGq8NX3iAyrcg9qJlroUk8 jFSCPrUHEFbQLY++44dgHg9RL60Gs3YzubhCXEXmkRas8AaJH3o7oDA/1Bs+qYRSktOx u6Zw9KmwNApauP9/gTsAJGolfxfZKBm1onVSQ/TJ7l+FbaB+P6jlQoYocxSFFfn9pCDp tJhA== X-Forwarded-Encrypted: i=1; AJvYcCVUACxGFDayfKK+zJZeiVhFUk+goC4PKrjjqxJvD6vlFkV6/BC1KnSdLYOvc52k0swrd5CklmYsak4LlmJy@postgresql.org X-Gm-Message-State: AOJu0Ywj6cBp3irwxYP4gNQZr4fOH74a+oPxI92BIi2gIBmhCawH6Kkq TyFFAGLo94ZyxDAXNkHXA2Bhhnxtay57lqlsaYKz+PVnIKWzHzJASgfePSO6yapSgiiZbzQg7DO ZYwKdvXbWhj+ntMce38OdlrqK9hmCFzc= X-Gm-Gg: ASbGncsfopXCcXQXhVrl/VUQ7daRuWRgiS/QFo2oZM8HIDp6c4spdAeAemyMj3nuMGW I2dOwgFgNI+mTVYGAIx4NHXUqmD4HTlg+mlK4l2nOdLyVLnQRGpM2xXUYslrjWsg/qOmO7i9a1Z aquQiWqAPpOg04DEN6P6hOz4OwuNCUP1szP0biA0Kq+itNVHULsLtTsuyLuRdn7R5JNnfT0cLly TsqK8SZRMtIj0DqxHAmqKrvvBxIEnm2pWu/mpsHRAZm9mAtcfSJUA== X-Google-Smtp-Source: AGHT+IH6Jj7s+kN2RnFLRTE8u16gKyl29m0NFQjPCGDKG72nY84jlnAg/ZWZ8evg0J5c5xeeUz8q571zHd8zRZsPo/Q= X-Received: by 2002:a05:690e:154d:10b0:63c:e0bb:2e8c with SMTP id 956f58d0204a3-63ce0bb3a7cmr471541d50.5.1759997239686; Thu, 09 Oct 2025 01:07:19 -0700 (PDT) MIME-Version: 1.0 References: <87il22cj51.fsf@163.com> In-Reply-To: From: Richard Guo Date: Thu, 9 Oct 2025 17:07:08 +0900 X-Gm-Features: AS18NWDJPNXNtffUKRYdu6c6rU8AbmoPTZrbQUNrjuKXXim0mwH7v_MsKE3ab_g Message-ID: Subject: Re: Eager aggregation, take 3 To: David Rowley Cc: Robert Haas , Tom Lane , Tender Wang , Paul George , Andy Fan , PostgreSQL-development , pgsql-hackers@lists.postgresql.org, Matheus Alcantara Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On Thu, Oct 9, 2025 at 10:49=E2=80=AFAM Richard Guo wrote: > On Wed, Oct 8, 2025 at 8:14=E2=80=AFPM David Rowley wrote: > > +-- Enable eager aggregation, which by default is disabled. > > +SET enable_eager_aggregate TO on; > > > The above comment and command mismatch to my understanding from > > looking at postgresql.conf.sample and guc_parameters.dat. > Right. This GUC was disabled by default prior to v17, and this is a > leftover from that. Will push a fix. Thanks for pointing it out! I noticed an unnecessary header include in initsplan.c. Will fix that as well. - Richard