[GitHub] [druid] clintropolis opened a new pull request #9989: fix NilVectorSelector filter optimization

[GitBox <gi...@apache.org>]

clintropolis opened a new pull request #9989:
URL: https://github.com/apache/druid/pull/9989


   Fixes an optimization to allow filters on `NilVectorSelector` to use `BooleanVectorValueMatcher` for things like a selector that matches null. The bug does not cause incorrect results, it just wasn't taking an optimization which it could have been using. 
   
   This was hard to functionally test with a query since the results were still correct, so I just added a test for this in the relevant method in `VectorMatchers`, `toBooleanIfPossible`.
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

[GitHub] [druid] clintropolis merged pull request #9989: fix NilVectorSelector filter optimization

[GitBox <gi...@apache.org>]

clintropolis merged pull request #9989:
URL: https://github.com/apache/druid/pull/9989


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org