[GitHub] [iceberg] aokolnychyi commented on a diff in pull request #5576: API: Extend FileIO in optional interfaces

[GitBox <gi...@apache.org>]

aokolnychyi commented on code in PR #5576:
URL: https://github.com/apache/iceberg/pull/5576#discussion_r949685046


##########
api/src/main/java/org/apache/iceberg/io/CredentialSupplier.java:
##########
@@ -26,7 +26,7 @@
  * the configured credential as a string, and use the credential for file access via other IO
  * libraries.
  */
-public interface CredentialSupplier {
+public interface CredentialSupplier extends FileIO {

Review Comment:
   All these interfaces assume they are mixed-in by `FileIO` instances (see docs).



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org