rdblue opened a new pull request #1730: URL: https://github.com/apache/iceberg/pull/1730
This fixes a `NullPointerException` that is thrown by a `ManifestReader` for delete files when there is a query filter. The `DeleteFileIndex` projects all fields of a delete manifest, so it doesn't call `select` to select specific columns, unlike `ManifestGroup`, which selects `*` by default. When select is not called, methods that check whether to add stats columns fail, but only if there is a row filter because stats columns are not needed if there is no row filter. Existing tests either called `select` to configure the reader, or didn't pass a row filter and projected all rows. This adds a test that uses `DeleteFileIndex` and a test for `ManifestReader`. This also fixes `dropStats` in addition to `requireStatsProjection`. Closes #1726, #1725. Co-authored-by: 钟保罗 <[email protected]> ---------------------------------------------------------------- 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]
