waynexia commented on pull request #8856: URL: https://github.com/apache/arrow/pull/8856#issuecomment-748494258
> I suggest that we keep the list_sort in the sort kernel for now. Hi @nevi-me, thanks for your reply. From my understanding, `list_sort` is not implemented in this pr. To avoid ambiguity, let me say that `list_sort` stands for an operator in `DataFusion`, and this pr implements the `ListArray` part of the `sort` kernel. Am I right @jorgecarleitao? > We can later move all ListArray-specific functions to a separate module. I agree with 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. For queries about this service, please contact Infrastructure at: [email protected]
