ajantha-bhat commented on PR #12629: URL: https://github.com/apache/iceberg/pull/12629#issuecomment-2841034055
I tried to move `PartitionStatsHandler` to `core` and use `InternalData`, I can't make test case run as it needs `iceberg-parquet` dependencies. I tried adding this to core test dependency. It will be a circular dependency. So, not allowed. Other way is to move `PartitionStatsUtil` to `data` module, but since computing stats is core functionality, I don't want to move it to data. So, we need to drop the idea of this movement. We need to work with having `PartitionStatsUtil` in core and `PartitionStatsHandler` in data, what can be done to keep interface simple. I will work it and also finish the compaction test. -- 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
