pvary commented on PR #12629: URL: https://github.com/apache/iceberg/pull/12629#issuecomment-2783731812
> > Do we expect someone just using the calculation and someone using the stat file generation too? Or it was just a technical limitation? > > @pvary, I think the original design tried to separate the functionality of stats evaluation and stats file management (stats read/write). However, I wouldn't expect someone to use `PartitionStatsUtil` directly. see #11216 After discussing this with @deniskuzZ, I feel that we probably would want to move the `PartitionStatsUtil` and the `PartitionStatsHandler` into the same package, and deprecate the methods from the `PartitionStatsUtil` (make them package private in the long run), to decrease the size of the API. What do you think @ajantha-bhat, @gaborkaszab? -- 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
