| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Use incremental sort paths for window functions |
| Date: | 2020-07-08 04:57:21 |
| Message-ID: | CAApHDvoOHobiA2x13NtWnWLcTXYj9ddpCkv9PnAJQBMegYf_xw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Over on [1] someone was asking about chained window paths making use
of already partially sorted input. (The thread is on -general, so I
guessed they're not using PG13.)
However, On checking PG13 to see if incremental sort would help their
case, I saw it didn't. Looking at the code I saw that
create_window_paths() and create_one_window_path() don't make any use
of incremental sort paths.
I quickly put together the attached. It's only about 15 mins of work,
but it seems worth looking at a bit more for some future commitfest.
Yeah, I'll need to add some tests as I see nothing failed by changing
this.
I'll just park this here until then so I don't forget.
David
| Attachment | Content-Type | Size |
|---|---|---|
| incremental_sort_for_windowpaths.patch | application/octet-stream | 2.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amul Sul | 2020-07-08 05:44:40 | Re: Proposal: Automatic partition creation |
| Previous Message | Fabien COELHO | 2020-07-08 04:53:52 | Re: Proposal: Automatic partition creation |