alamb opened a new pull request, #2645: URL: https://github.com/apache/arrow-datafusion/pull/2645
# Which issue does this PR close? re : https://github.com/apache/arrow-datafusion/issues/2427 # Rationale for this change `SortKeyCursor::compare` is the bottleneck for merge performance (details on https://github.com/apache/arrow-datafusion/issues/2427#issuecomment-1140425072) As I prepare to improve the performance of this code (and make it more complicated), I want it more isolated and independent so I can: 1. Understand the code better 2. Validate my approach 3. Ensure I dont introduce performance # What changes are included in this PR? 1. Move `SortKeyCursor` into its own module 2. Move `RowIndex` into its own module 3. add `sort_key_cursor` test # Are there any user-facing changes? No # Does this PR break compatibility with Ballista? 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]
