[GitHub] jihoonson commented on issue #5958: Part 2 of changes for SQL Compatible Null Handling

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

jihoonson commented on issue #5958: Part 2 of changes for SQL Compatible Null Handling
URL: https://github.com/apache/incubator-druid/pull/5958#issuecomment-408538927
 
 
   Thanks @nishantmonu51. Would you please check this comment: https://github.com/apache/incubator-druid/pull/5958#discussion_r204937099?
   
   Also, I'm not sure what have done in https://github.com/apache/incubator-druid/pull/5278 + this PR and not. I think at least `Comparisons between null and non-null values will follow three-valued logic and the result of a comparison involving NULL is not a boolean value – it is a null value.` is not done yet? Would you please clarify what are remaining in https://github.com/apache/incubator-druid/issues/4349?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

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