On Thu, May 12, 2022 at 10:38:42AM +1200, David Rowley wrote:
> On Wed, 11 May 2022 at 14:38, Justin Pryzby <pry...@telsasoft.com> wrote:
> > I wonder if this is also relevant.
> >
> > 65014000b35 Replace polyphase merge algorithm with a simple balanced k-way 
> > merge.
> 
> Thanks for highlighting that. It very much is relevant. In fact, it
> seems to account for most of the 25% I mentioned.  That particular
> test was sorting 10 million tuples with 4MB of work_mem.
> 
> I think that "Improve sorting performance (Heikki Linnakangas)" should
> be moved out from "E.1.3.1.2. Indexes" and put below "E.1.3.1.4.
> General Performance"

Yes, good point, moved.

> The text likely should include the words "disk-based" so that it's
> clear that it's not the same as the other line about "in-memory
> sorts".  I'd also be open to just having a single line too. I'd vote
> to put Heikki's name first if we did that.
> 
> Maybe:
> 
> * Improve performance of sorting tuples (Heikki Linnakangas, Ronan
> Dunklau, David Rowley, Thomas Munro)
> 
> This improves the merging performance of individual on-disk sort
> batches, reduces memory consumption for in-memory sorts and reduces
> CPU overheads for certain in-memory sorts.

I kept separate entries:

        <!--
        Author: Heikki Linnakangas <heikki.linnakan...@iki.fi>
        2021-10-18 [65014000b] Replace polyphase merge algorithm with a simple 
balanced
        Author: Heikki Linnakangas <heikki.linnakan...@iki.fi>
        2021-10-25 [166f94377] Clarify the logic in a few places in the new 
balanced me
        -->
        
        <listitem>
        <para>
        Improve performance for sorts that exceed work_mem (Heikki Linnakangas)
        </para>
        
        <para>
        Specifically, switch to a batch sorting algorithm that uses more
        output streams internally.
        </para>
        </listitem>
        
        <!--
        Author: David Rowley <drow...@postgresql.org>
        2021-07-22 [91e9e89dc] Make nodeSort.c use Datum sorts for single 
column sorts
        Author: David Rowley <drow...@postgresql.org>
        2022-04-04 [40af10b57] Use Generation memory contexts to store tuples 
in sorts
        Author: John Naylor <john.nay...@postgresql.org>
        2022-04-02 [697492434] Specialize tuplesort routines for different 
kinds of abb
        -->
        
        <listitem>
        <para>
        Improve performance and reduce memory consumption of in-memory
        sorts (Ronan Dunklau, David Rowley, Thomas Munro)
        </para>
        </listitem>

-- 
  Bruce Momjian  <br...@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Indecision is a decision.  Inaction is an action.  Mark Batterson



Reply via email to