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

Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/04/19 20:57:27 UTC

[GitHub] [iceberg] yyanyy commented on issue #2492: NullPointerException on containsNan()

yyanyy commented on issue #2492:
URL: https://github.com/apache/iceberg/issues/2492#issuecomment-822779814


   Sorry for the late response, I think you are right. I think we should default it to true if this value is missing, meanwhile I think we may need to modify the other parts of this class to account for the possibility that min/max could be NaN and thus comparison with the input value produces incorrect outcome. I'll send out a PR today. Thanks for looking into it! 


-- 
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: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org