ajantha-bhat commented on PR #7105: URL: https://github.com/apache/iceberg/pull/7105#issuecomment-1709488967
I like the strong case by @flyrain (https://github.com/apache/iceberg/pull/7105#discussion_r1317716670) and @szehon-ho (https://github.com/apache/iceberg/pull/7105#discussion_r1318005645) to retain delete stats. Very good use cases. So, I am adding back the delete stats. (Yeah it is too much back and forth on this 🤦 ). I hope @rdblue also agree this time on keeping delete stats. Now, about the accurate record count, we can have a new field as `total_record_count` initalized to -1 and will be filled only during the async call (via CALL procedure). The sync call (writers writing them), can skip computing this as we discussed that writers may not have this information and may need IO of all positional and equality delete files which is expensive. Let me know what everyone thinks on this. -- 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]
