MIME-Version: 1.0
References: <CAMbWs48jzLrPt1J_00ZcPZXWUQKawQOFE8ROc-ADiYqsqrpBNw@mail.gmail.com>
 <87il22cj51.fsf@163.com> <CAMbWs49=eAd2W9jCtGhaZPPp+SOC_2rg16RTG74xAht=hkr5JQ@mail.gmail.com>
 <CAMbWs49Nc4M3H+eCf1+8w8piDyEECjRb-gK_JMF4VvcyWwGEVQ@mail.gmail.com>
 <CAMbWs49E_dR0nobsExsyetpnBpHObLTsQLsEbWKQLkh0omPxNg@mail.gmail.com>
 <CAMbWs49B_qUiHvu2EqLHZRpLr3p_+QPBs50n2=L5ibYzniwTzA@mail.gmail.com>
 <CAMbWs48KCQtDymnYi4M=Vz+WMzo3fkBxffJsyk6VX6hOXXv+VA@mail.gmail.com>
 <CAMbWs49sv_MuOYqqrtmBN_oYf8VSQ2BXDwXaTpJTn_YfwyYdWQ@mail.gmail.com>
 <CAMbWs49U8Sddx_fGszPdvA3jp_nheynxaqm5Y4NqMV21VBYAuQ@mail.gmail.com>
 <CAMbWs4-LwyOg9ga+NVF7yQbMi0ZsZdN1G_sO2v=YJHV18=19+A@mail.gmail.com>
 <CALA8mJquG_zCJXfVwash5LKqHGtZXQmq7RfTSaRDUzGYeW=7Rw@mail.gmail.com>
 <CAMbWs4_EjgcBib5+y1LYcGB3EK3Y6R+OOxGKfJo42fDovadk1g@mail.gmail.com>
 <CALA8mJqe0anNM8_V6cOeOQnCHUTQggn7iOQNyQr1VaN_xMjz+w@mail.gmail.com>
 <CAMbWs48eE-s-jCicC8pSVfXk8Ws-ZvUKnsw8qH-DkVBdYv0eJQ@mail.gmail.com> <CAMbWs483a7-8M0pDttG44r-+8Gevn9VG0xNceE3WpkEQxJXPZw@mail.gmail.com>
In-Reply-To: <CAMbWs483a7-8M0pDttG44r-+8Gevn9VG0xNceE3WpkEQxJXPZw@mail.gmail.com>
From: Tender Wang <tndrwang@gmail.com>
Date: Wed, 4 Sep 2024 11:48:30 +0800
Message-ID: <CAHewXNmwdmf2DDYVfqySYKQ7DnxyUUDnaaUa6+bdfF-oUAh_Vg@mail.gmail.com>
Subject: Re: Eager aggregation, take 3
To: Richard Guo <guofenglinux@gmail.com>
Cc: Paul George <p.a.george19@gmail.com>, Andy Fan <zhihuifan1213@163.com>, 
	PostgreSQL-development <pgsql-hackers@postgresql.org>, pgsql-hackers@lists.postgresql.org
Content-Type: multipart/alternative; boundary="000000000000d1b0ba0621430c9b"
Archived-At: <https://www.postgresql.org/message-id/CAHewXNmwdmf2DDYVfqySYKQ7DnxyUUDnaaUa6%2BbdfF-oUAh_Vg%40mail.gmail.com>
Precedence: bulk

--000000000000d1b0ba0621430c9b
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Richard Guo <guofenglinux@gmail.com> =E4=BA=8E2024=E5=B9=B48=E6=9C=8821=E6=
=97=A5=E5=91=A8=E4=B8=89 15:11=E5=86=99=E9=81=93=EF=BC=9A

> On Fri, Aug 16, 2024 at 4:14=E2=80=AFPM Richard Guo <guofenglinux@gmail.c=
om>
> wrote:
> > I had a self-review of this patchset and made some refactoring,
> > especially to the function that creates the RelAggInfo structure for a
> > given relation.  While there were no major changes, the code should
> > now be simpler.
>
> I found a bug in v10 patchset: when we generate the GROUP BY clauses
> for the partial aggregation that is pushed down to a non-aggregated
> relation, we may produce a clause with a tleSortGroupRef that
> duplicates one already present in the query's groupClause, which would
> cause problems.
>
> Attached is the updated version of the patchset that fixes this bug
> and includes further code refactoring.
>

The v11-0002 git am failed on HEAD(6c2b5edecc).

tender@iZ2ze6la2dizi7df9q3xheZ:/workspace/postgres$ git am
v11-0002-Implement-Eager-Aggregation.patch
Applying: Implement Eager Aggregation
error: patch failed: src/test/regress/parallel_schedule:119
error: src/test/regress/parallel_schedule: patch does not apply
Patch failed at 0001 Implement Eager Aggregation
hint: Use 'git am --show-current-patch=3Ddiff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".


--=20
Thanks,
Tender Wang

--000000000000d1b0ba0621430c9b
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote">=
<div dir=3D"ltr" class=3D"gmail_attr">Richard Guo &lt;<a href=3D"mailto:guo=
fenglinux@gmail.com">guofenglinux@gmail.com</a>&gt; =E4=BA=8E2024=E5=B9=B48=
=E6=9C=8821=E6=97=A5=E5=91=A8=E4=B8=89 15:11=E5=86=99=E9=81=93=EF=BC=9A<br>=
</div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;b=
order-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Aug 16, 202=
4 at 4:14=E2=80=AFPM Richard Guo &lt;<a href=3D"mailto:guofenglinux@gmail.c=
om" target=3D"_blank">guofenglinux@gmail.com</a>&gt; wrote:<br>
&gt; I had a self-review of this patchset and made some refactoring,<br>
&gt; especially to the function that creates the RelAggInfo structure for a=
<br>
&gt; given relation.=C2=A0 While there were no major changes, the code shou=
ld<br>
&gt; now be simpler.<br>
<br>
I found a bug in v10 patchset: when we generate the GROUP BY clauses<br>
for the partial aggregation that is pushed down to a non-aggregated<br>
relation, we may produce a clause with a tleSortGroupRef that<br>
duplicates one already present in the query&#39;s groupClause, which would<=
br>
cause problems.<br>
<br>
Attached is the updated version of the patchset that fixes this bug<br>
and includes further code refactoring.<br></blockquote><div><br></div><div>=
The v11-0002 git am failed on HEAD(6c2b5edecc).</div><div><br></div>tender@=
iZ2ze6la2dizi7df9q3xheZ:/workspace/postgres$ git am v11-0002-Implement-Eage=
r-Aggregation.patch<br>Applying: Implement Eager Aggregation<br>error: patc=
h failed: src/test/regress/parallel_schedule:119<br>error: src/test/regress=
/parallel_schedule: patch does not apply<br>Patch failed at 0001 Implement =
Eager Aggregation<br>hint: Use &#39;git am --show-current-patch=3Ddiff&#39;=
 to see the failed patch<br>When you have resolved this problem, run &quot;=
git am --continue&quot;.<br>If you prefer to skip this patch, run &quot;git=
 am --skip&quot; instead.<br><div>To restore the original branch and stop p=
atching, run &quot;git am --abort&quot;.</div><div>=C2=A0</div></div><br cl=
ear=3D"all"><div><br></div><span class=3D"gmail_signature_prefix">-- </span=
><br><div dir=3D"ltr" class=3D"gmail_signature"><div dir=3D"ltr"><div>Thank=
s,</div><div>Tender Wang</div></div></div></div>

--000000000000d1b0ba0621430c9b--