fallintoplace commented on issue #1091: URL: https://github.com/apache/iceberg-go/issues/1091#issuecomment-4491458644
Yes, happy to take this. I’ll keep the PR focused: make strict null handling conservative when null metrics are missing, while preserving the required-field fast path where it is safe. I’ll add the regression test Tanmay @tanmayrauth described for a nullable field with bounds present but no null count, where `id > 0` should not strictly match the whole file. For the NaN-count side: do you prefer I include the analogous floating-point handling in the same PR if it’s localized, or keep the first PR strictly to missing null counts and follow up separately? -- 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]
