ajantha-bhat commented on issue #10425: URL: https://github.com/apache/iceberg/issues/10425#issuecomment-2144294227
I think initially it was designed that `ManifestFiles` will be local to `iceberg-core` module and we can have a public API's around it that are exposed to other modules. While working on partition stats at `iceberg-data` module, I had to add some functionality in `iceberg-core` module to transform manifests and use it in `iceberg-data` module. But in your case, if it is a custom transformation (not required for other iceberg modules), it makes sense to expose `ManifestFiles`. So, I am generally +1 on this idea. Lets see what others think. -- 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
