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 2022/11/26 19:06:57 UTC

[GitHub] [arrow-rs] viirya commented on pull request #3197: Add dictionary suppport to like, ilike, nlike, nilike kernels

viirya commented on PR #3197:
URL: https://github.com/apache/arrow-rs/pull/3197#issuecomment-1328098768

   > We don't seem to provide `eq_dict` or `gt_dict` instead this is hidden within `eq_dyn`, perhaps we should do the same here?
   
   I think it is because there are `like_dict_scalar`, `nlike_dict_scalar` etc for dictionary arrays. I agree that for other comparison kernels, only primitive and _dyn kernels are available. Maybe we should remove these `_dict_scalar` and only keep `_dyn` for dictionary support?


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