alamb opened a new issue, #2990:
URL: https://github.com/apache/arrow-rs/issues/2990

   **Describe the bug**
   Found by @Dandandan  and @isidentical while testing the arrow 26.0.0 upgrade 
in DataFusion
   
   https://github.com/apache/arrow-datafusion/pull/4039#issuecomment-1296752289
   
   
   **To Reproduce**
   Run `lexsort_to_indices`  with a `limit` on a multi-column sort. 
   
   
   **Expected behavior**
   <!--
   A clear and concise description of what you expected to happen.
   -->
   
   **Additional context**
   quoting @isidentical: 
   
   >  Probably due to the changes in 
https://github.com/apache/arrow-rs/pull/2929, where we previously created an 
array of (&value_indices)[0..len] but now just create it from value_indices 
directly without any limit (the single-column case is handled separately; and 
it still respects the limit).
   
   @tustvold  authored the PR in question and I approved them so I feel 
responsible for fixing this


-- 
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: github-unsubscr...@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to