zeroshade commented on issue #66: URL: https://github.com/apache/arrow-go/issues/66#issuecomment-4185096887
> What I'm thinking for the kernel is: Agree completely with everything you have here! One recommendation is to look at the [c++ arrow sort_indices implementation](https://github.com/apache/arrow/blob/main/cpp/src/arrow/compute/kernels/vector_sort.cc) for inspiration as the compute package here is based on the C++ compute package. > Do we also want public methods like the following? Don't forget `SortIndices(Datum)` and `Sort(Datum)` which would just dispatch to `CallFunction` appropriately. -- 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]
