Re: [Patch] Build the heap more efficient in tuplesort.c

2025-11-30 Thread cca5507
Hi, > 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 th

Re: [Patch] Build the heap more efficient in tuplesort.c

2025-11-30 Thread David Rowley
On Sun, 30 Nov 2025 at 22:36, cca5507 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 binaryhe

[Patch] Build the heap more efficient in tuplesort.c

2025-11-30 Thread cca5507
Hi hackers, 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? -- Regards, ChangAo