jorgecarleitao edited a comment 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
I would place this operator on a separate location, i.e. not in `sort`, to
avoid confusion with the operator that sorts the array.
----------------------------------------------------------------
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:
[email protected]
