Re: [Patch] Build the heap more efficient in tuplesort.c - Mailing list pgsql-hackers

From David Rowley
Subject Re: [Patch] Build the heap more efficient in tuplesort.c
Date
Msg-id CAApHDvoDVU8QW3+31QAJ-owfD4=pJPFRDVtOS=QrBXrU2+26sA@mail.gmail.com
Whole thread Raw
In response to [Patch] Build the heap more efficient in tuplesort.c  ("cca5507" <cca5507@qq.com>)
Responses Re: [Patch] Build the heap more efficient in tuplesort.c
List pgsql-hackers
On Sun, 30 Nov 2025 at 22:36, cca5507 <cca5507@qq.com> wrote:
> Now we build the heap by using tuplesort_heap_insert(), which has a sift-up every call.
>
> To make it more efficient, I want to add tuplesort_heap_insert_unordered() and tuplesort_heap_build()
> just like binaryheap_add_unordered() and binaryheap_build().
>
> Thoughts?

For performance patches, you should include example workloads that
your patch speeds up. Include benchmark results with and without your
patch. Demonstrate you've not regressed any other workloads at the
expense of the ones you intend to speed up.

It's not up to reviewers to do this for you.

David



pgsql-hackers by date:

Previous
From: Jelte Fennema-Nio
Date:
Subject: Re: Early December Commitfest app release
Next
From: "Hayato Kuroda (Fujitsu)"
Date:
Subject: RE: [buildfarm related] Machines gcc experimental failed test_lfind