You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "yevgenypats (via GitHub)" <gi...@apache.org> on 2023/03/22 17:48:40 UTC

[GitHub] [arrow] yevgenypats opened a new issue, #34690: [Go][Table] Implement sort function

yevgenypats opened a new issue, #34690:
URL: https://github.com/apache/arrow/issues/34690

   ### Describe the enhancement requested
   
   Seems the python library has a sort functionality for table - https://arrow.apache.org/cookbook/py/data.html#sort-a-table
   
   Will it make sense to implement the same for the Go library (unless it is already implemented and I missed it)?
   
   ### Component(s)
   
   Go


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow] zeroshade commented on issue #34690: [Go][Table] Implement sort function

Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade commented on issue #34690:
URL: https://github.com/apache/arrow/issues/34690#issuecomment-1480136712

   It definitely makes sense to implement the same functionality for Go and it's on my list of stuff I intend to get to, but if you wanna try your hand at it, that would be awesome. :smile:
   
   We already have a `Take` kernel in the Go compute that works with Tables (and Records, ChunkedArrays and Arrays) so you'd just need to first implement a `sort_indices` vector-kernel which would output the list of indexes for ordering things (like the Acero C++ `sort_indices` kernel). After that is implemented and able to take a Table, it's a fairly simple combination of operations => `Take(table, sort_indices(table)) == sorted table`


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] zeroshade commented on issue #34690: [Go][Table] Implement sort function

Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade commented on issue #34690:
URL: https://github.com/apache/arrow/issues/34690#issuecomment-1481393029

   @hermanschaaf Sounds great! Feel free to file a draft PR and tag me to ask for help if you need or if you have any questions


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] hermanschaaf commented on issue #34690: [Go][Table] Implement sort function

Posted by "hermanschaaf (via GitHub)" <gi...@apache.org>.
hermanschaaf commented on issue #34690:
URL: https://github.com/apache/arrow/issues/34690#issuecomment-1481391529

   I'd like to take a look at this. It'll be my first contribution to Arrow and I might need some pointers, but I'll give it a shot. 😄 


-- 
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: github-unsubscribe@arrow.apache.org

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