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 2020/01/27 23:40:36 UTC

[GitHub] [druid] clintropolis commented on issue #9251: fix some issues with filters on numeric columns with nulls

clintropolis commented on issue #9251: fix some issues with filters on numeric columns with nulls
URL: https://github.com/apache/druid/pull/9251#issuecomment-579009125
 
 
   >Another comment: did you consider removing the default impl of applyNull from the DruidXPredicate interfaces? It seems like it being missing is just asking for null handling bugs.
   
   Yes, and I agree, though since I was planning to look at expression filters and the others as a follow-up I was considering making this change later. Should I just do it all now?

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


With regards,
Apache Git Services

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