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 > SQL also applies NULL handling to all functions and operators: in general, f(NULL) -> NULL for every function f, unless special cased, as in the logical operators. I didn't do a complete survey, but i believe many functions are doing the correct thing with regards to null handling, (but it would probably be worth validating and correcting any behavior that isn't consistent) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
