[GitHub] [arrow] jorgecarleitao commented on pull request #8856: Arrow-10355: [Rust] Add support for list_sort

[GitBox <gi...@apache.org>]

jorgecarleitao commented on pull request #8856:
URL: https://github.com/apache/arrow/pull/8856#issuecomment-739675217


   Hi @waynexia . Thanks a lot for this. I think that the idea of a list_sort is a bit different: in the test you wrote, it would correspond to:
   
   ```
               vec![
                   Some(vec![1, 0]),
                   Some(vec![4, 3, 2, 1]),
                   Some(vec![2, 3, 4]),
                   Some(vec![3, 3, 3, 3]),
                   Some(vec![1, 1]),
               ],
               Some(SortOptions {
                   descending: false,
                   nulls_first: false,
               }),
               vec![
                   Some(vec![0, 1]),
                   Some(vec![1, 2, 3, 4]),
                   Some(vec![2, 3, 4]),
                   Some(vec![3, 3, 3, 3]),
                   Some(vec![1, 1]),
               ],
   ```
   
   i.e. it is a operator that operates on the individual entries of the array, and sorts them individually. This would correspond to the `array_sort` operator in Spark: https://spark.apache.org/docs/latest/api/sql/index.html#array_sort
   
   
   
   
   


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

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