johanpel edited a comment on pull request #11883:
URL: https://github.com/apache/arrow/pull/11883#issuecomment-988626282
Thanks for the comments. I added the docs. I also added a test because the
function can be applied to String/LargeString as well. I'm in doubt whether we
would want to prevent applying this function to String/LargeString. Use cases
would be quite exotic.
If we'd like to prevent it I would have to introduce `BinaryTypes()` with
`{binary(), large_binary()}` only in `arrow/type.h` and use that to add the
kernel to the registry only for these types.
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
