ianmcook commented on pull request #8990: URL: https://github.com/apache/arrow/pull/8990#issuecomment-809650002
Regarding "join" in the name: +1 for consistency with Python's join function -1 because as Arrow gains more database-like features, the word "join" is likely to confuse people. "concat" might be better. Regarding "binary" in the name: IMO that is likely to cause users to get the impression that this is _not_ for strings. It would also be good for the name to reflect that this operates on a list array. How about renaming it `list_concat`? -- 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]
