findepi commented on PR #5795: URL: https://github.com/apache/iceberg/pull/5795#issuecomment-1262030928
Applied or responded to comments. I didn't do anything with the public Table API yet. I think, however, it shouldn't be a blocker, since the affected `ReachableFileUtil` class already depends on non-API information in other methods. We can improve `ReachableFileUtil` after https://github.com/apache/iceberg/pull/4741, or we can land https://github.com/apache/iceberg/pull/4741 first and improve here. @rdblue please take another look. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
