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/04/20 07:16:55 UTC

[GitHub] [arrow] pitrou commented on pull request #12891: ARROW-12659: [C++] Support is_valid as a guarantee

pitrou commented on PR #12891:
URL: https://github.com/apache/arrow/pull/12891#issuecomment-1103552544

   > Hmm. I guess we are treating guarantees and filters differently. `x < 10` as a guarantee implies `is_valid(x)`, but not as a filter. We may want to fix that, but that would also be a drastic change.
   
   I think it would definitely be worthwhile to fix it. It's delicate enough to think about simplifications without this oddity.
   
   Also, isn't it surprising as a user for the `x < 10` filter to accept nulls? It wouldn't in SQL for example.


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