Re: Eager aggregation, take 3

From: Richard Guo <guofenglinux(at)gmail(dot)com>
To: Paul George <p(dot)a(dot)george19(at)gmail(dot)com>
Cc: Andy Fan <zhihuifan1213(at)163(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Eager aggregation, take 3
Date: 2024-08-16 08:14:41
Message-ID: CAMbWs48eE-s-jCicC8pSVfXk8Ws-ZvUKnsw8qH-DkVBdYv0eJQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

Attached is the updated version of the patchset. Previously, the
patchset was not well-split, which made it time-consuming to
distribute the changes across the patches during the refactoring. So
I squashed them into two patches to save effort.

Thanks
Richard

Attachment Content-Type Size
v10-0001-Introduce-RelInfoList-structure.patch application/octet-stream 14.8 KB
v10-0002-Implement-Eager-Aggregation.patch application/octet-stream 162.0 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Aleksander Alekseev 2024-08-16 08:41:37 Re: [PATCH] Add get_bytes() and set_bytes() functions
Previous Message Yugo Nagata 2024-08-16 07:49:36 Re: [PATCH] Add get_bytes() and set_bytes() functions