huaxingao commented on PR #9721: URL: https://github.com/apache/iceberg/pull/9721#issuecomment-1957806734
@aokolnychyi Thanks a lot for your feedback! I agree that the current approach exposes a lot of the internal classes, and injecting a custom `PartitionReaderFactory` would indeed offer a cleaner solution. However, this might require me to duplicate additional code on `Comet` side. Let me explore this strategy a bit further to see how it goes. -- 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
