Re: PoC: Partial sort
Alexander Korotkov <aekorotkov@gmail.com>
From: Alexander Korotkov <aekorotkov@gmail.com>
To: Marti Raudsepp <marti@juffo.org>
Cc: Robert Haas <robertmhaas@gmail.com>, Andreas Karlsson <andreas@proxel.se>,
David Rowley <dgrowleyml@gmail.com>,
pgsql-hackers <pgsql-hackers@postgresql.org>,
Andres Freund <andres@2ndquadrant.com>,
Martijn van Oosterhout <kleptog@svana.org>
Date: 2014-02-10T18:59:48Z
Lists: pgsql-hackers
Attachments
- partial-sort-basic-1.patch.gz (application/x-gzip) patch
- partial-sort-merge-1.patch.gz (application/x-gzip) patch
On Mon, Feb 10, 2014 at 2:33 PM, Marti Raudsepp <marti@juffo.org> wrote: > On Sun, Feb 9, 2014 at 7:37 PM, Alexander Korotkov <aekorotkov@gmail.com> > wrote: > > This is not only place that worry me about planning overhead. See > > get_cheapest_fractional_path_for_pathkeys. I had to estimate number of > > groups for each sorting column in order to get right fractional path. > > AFAICT this only happens once per plan and the overhead is O(n) to the > number of pathkeys? I can't get worried about that, but I guess it's > better to test anyway. > > PS: You didn't answer my questions about splitting the patch. I guess > I'll have to do that anyway to run the tests. > Done. Patch is splitted. ------ With best regards, Alexander Korotkov.
Commits
-
Implement Incremental Sort
- d2d8a229bc58 13.0 landed
-
Improve memory management for external sorts.
- 0011c0091e88 9.6.0 cited