szehon-ho commented on PR #4520: URL: https://github.com/apache/iceberg/pull/4520#issuecomment-1095624358
Adding an observation, the case where wrong results are filtered out is limited to V2 tables, where partition fields are removed (ie, in the initial description). In tests for any partition spec modification in V1 (in V1, fields are never removed), or V2 tables where partition fields are modified or added but not removed, we get a NPE in the ManifestEvaluator if we try to query files written by different partition spec than current one. Still worth to fix, just making a note for the impact. -- 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]
