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/12/20 06:42:14 UTC

[GitHub] [arrow] bkmgit commented on pull request #11882: ARROW-9843: [C++] Implement Between ternary kernel

bkmgit commented on pull request #11882:
URL: https://github.com/apache/arrow/pull/11882#issuecomment-997640514


   @cyb70289 @lidavidm A light review of this pull request would be helpful. Would like to add 
   - further tests 
   - function options to choose comparison mode
   Good comparison of strings is out of scope, but would be good to add in future, currently utf8 codepoint is used for string comparison which is ok for ascii only text, but poor for most other texts.


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