| 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: | Whole Thread | Raw Message | 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 |
| 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 |