gaborkaszab commented on PR #15979: URL: https://github.com/apache/iceberg/pull/15979#issuecomment-4282289244
Hi @hemanthboyina , I gave this PR some thought the recent days, and frankly I'm a bit hesitant here. The reason is that this stat we are populating now is a derived stat, so all the information is there in the existing stats to produce this new one. Additionally, I think the code needed for this isn't that simple either. I'm wondering how much sense it makes to use the Iceberg lib to produce this stat or rather we simply say that we leave it to the engines to calculate it on their own. I know this is part of the spec now, but maybe we'd like to revisit that. To get better coverage, would you mind raising this to the dev@ list? -- 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]
