You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/06/15 06:41:38 UTC

[GitHub] [arrow] jorisvandenbossche commented on pull request #10520: ARROW-12709: [C++] Add binary_join_element_wise

jorisvandenbossche commented on pull request #10520:
URL: https://github.com/apache/arrow/pull/10520#issuecomment-861221807


   > The use of "binary" in the names of these string join kernels is unfortunate; it's not clear at first glance whether "binary" is a reference to the arity or to the input type.
   
   Yeah, personally I would be fine to have an alias that uses "string" to have some more recognizable name ("string_join_element_wise/list"), but not sure we want to start adding aliases in general ..


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