You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/12/01 02:28:26 UTC

[GitHub] [druid] clintropolis edited a comment on pull request #11184: vectorize logical operators and boolean functions

clintropolis edited a comment on pull request #11184:
URL: https://github.com/apache/druid/pull/11184#issuecomment-983224594


   >Does the change also cover the negation operator? NOT a where a is NULL results in NULL. (Note that a IS NOT NULL is entirely different!)
   
   This was already behaving correctly in SQL compatible null handling mode, so no changes were needed
   
   > Did a prior (or will a future) change handle nulls in math operators? 10 + NULL -> NULL?
   
   As far as I know, these should also already behave correctly prior to this PR


-- 
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: commits-unsubscribe@druid.apache.org

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