Light-City opened a new pull request, #38584: URL: https://github.com/apache/arrow/pull/38584
### Rationale for this change support multi sortkey nulls first. ``` order by i nulls first, j, k nulls first; ``` The current null sorting only supports all sortkeys, not a certain sortkey, so NullPlacement is extended to the SortKey field. Since the underlying framework is very well written, when modifying this function, you only need to pass the null_placement of each SortKey in. That’s it. ### What changes are included in this PR? SortKey structure, NullPlacemnt transfer logic, sorting logic and Ording related, test related ### Are these changes tested? yes, I changed the code inside vector_sort_test.cc and performed additional tests. ### Are there any user-facing changes? yes, pg database include null sorting of multiple sort keys. -- 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]
