Light-City opened a new pull request, #39701:
URL: https://github.com/apache/arrow/pull/39701

   
   ### Rationale for this change
   
   just like https://github.com/StarRocks/starrocks/pull/14609
   
   we can use std::inplace_merge to replace merge.
   
   Since our indices are also a whole block of memory, after using 
std::inplace_merge, we can reduce the memory allocation of temp_indices and 
reduce std::copy operations, which has a natural advantage for us, so here I 
think std::inplace_merge is more suitable
   
   ### What changes are included in this PR?
   
   sort operator, vector_sort.cc
   
   ### Are these changes tested?
   
   yes, run vector_sort_test.cc
   
   ### Are there any user-facing changes?
   
   no.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to