pvary commented on PR #12629: URL: https://github.com/apache/iceberg/pull/12629#issuecomment-2783310921
> @pvary, i don't follow where you are proposing these API changes? > [..] > If you are proposing to implement these API in `PartitionStatsUtil`, it would be problematic since it doesn't have ref to data module, see [#12629 (comment)](https://github.com/apache/iceberg/pull/12629#discussion_r2013945601) I try to wrap my head around the API. Is there some specific reason to separate the 2 util classes? Do we expect someone just using the calculation and someone using the stat file generation too? Or it was just a technical limitation? (Left a comment on the discussion linked above) -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
