gaborkaszab opened a new pull request, #17825: URL: https://github.com/apache/iceberg/pull/17825
Reverts apache/iceberg#16125 Giving this another thought, I think we have to revert this because it's a behavioral change. If there are users building on positional delete file count, then this field is changing now. E.g. for V3 table previously the DV cost was included in that field, but will be zero after this PR. For instance, if a maintenance system is checking pos del file count and kicks maintenance if it reaches a threshold, it won't work after this PR. Even though I think the changes are logically correct, we can't have such a change to keep compatibility for existing users. -- 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]
