yyanyy edited a comment 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. Thanks for looking into it and raising the PR to fix 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
