[C++] Support STRING_VIEW and BINARY_VIEW in "array_take" and "array_filter" functions

apache / arrow

Apache Arrow is the universal columnar format and multi-language toolbox for fast data interchange and in-memory analytics

https://arrow.apache.org/

Apache License 2.0

14.48k stars 3.52k forks source link

[C++] Support STRING_VIEW and BINARY_VIEW in "array_take" and "array_filter" functions #43010

Open felipecrv opened 4 months ago

felipecrv commented 4 months ago

Describe the bug, including details regarding any error messages, version, and platform.

Kernels for Type::STRING_VIEW/BINARY_VIEW are missing.

Component(s)

C++

mapleFU commented 3 months ago

@felipecrv Hi if you're busy, I may take time to do this, since during testing read/write utf8 it's better to having some useful tools. I may not familiar with compute but I'd like to have a try

felipecrv commented 3 months ago

@mapleFU you can go ahead and make a simple implementation (similar to the one for strings), but note that I'm about to merge #41700 which moves a lot of stuff around take and filter while also introducing ways of implementing more efficient take kernels.