a-reich opened a new pull request, #41495:
URL: https://github.com/apache/arrow/pull/41495

   ### Rationale for this change
   Closes issue https://github.com/apache/arrow/issues/41464. Fix 
`StructArray.sort` method's `by` param to work in the case of `by=None` which 
was documented to mean sort by all fields (the default), but would raise an 
exception.
   
   ### What changes are included in this PR?
   * Add a unit test with by=None in `test_struct_array_sort` that fails on main
   * Fix the sort method
   
   ### Are these changes tested?
   yes
   
   ### Are there any user-facing changes?
   yes


-- 
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