| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | Richard Guo <guofenglinux(at)gmail(dot)com> |
| Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: An improvement on parallel DISTINCT |
| Date: | 2024-02-07 08:24:08 |
| Message-ID: | CAApHDvrGpb2Yej9Z-dKLzQP9kxsR4uM_wBmYv34-tRRnDUa0GQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, 5 Feb 2024 at 14:42, Richard Guo <guofenglinux(at)gmail(dot)com> wrote:
>
>
> On Fri, Feb 2, 2024 at 7:36 PM David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
>> I think we should just make it work the same way as
>> create_grouping_paths(), where grouping_target is passed as a
>> parameter.
>>
>> I've done it that way in the attached.
>
>
> The change looks good to me.
I pushed the PathTarget changes.
> BTW, I kind of doubt that 'create_partial_distinct_paths' is a proper
> function name given what it actually does.
I didn't make any changes here. I don't think messing with this is
worth the trouble.
David
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bharath Rupireddy | 2024-02-07 08:34:39 | Re: Printing backtrace of postgres processes |
| Previous Message | Peter Eisentraut | 2024-02-07 08:18:29 | pg_get_expr locking |