jackye1995 commented on code in PR #7562:
URL: https://github.com/apache/iceberg/pull/7562#discussion_r1188007551
##########
aws/src/main/java/org/apache/iceberg/aws/s3/S3FileIOProperties.java:
##########
@@ -324,6 +341,30 @@ public class S3FileIOProperties implements Serializable {
public static final boolean PRELOAD_CLIENT_ENABLED_DEFAULT = false;
+ /**
+ * Configure the AWS credentials provider used to create AWS clients. A
fully qualified concrete
+ * class with package that implements the {@link AwsCredentialsProvider}
interface is required.
+ *
+ * <p>Additionally, the implementation class must also have a create() or
create(Map) method
+ * implemented, which returns an instance of the class that provides aws
credentials provider.
+ *
+ * <p>Example:
+ *
client.credentials-provider=software.amazon.awssdk.auth.credentials.SystemPropertyCredentialsProvider
+ *
+ * <p>When set, the default client factory {@link
+ * org.apache.iceberg.aws.AwsClientFactories#defaultFactory()} and S3 client
factory class will
+ * use this provider to get AWS credentials provided instead of reading the
default credential
+ * chain to get AWS access credentials.
+ */
+ public static final String CLIENT_CREDENTIALS_PROVIDER =
"client.credentials-provider";
Review Comment:
this should not be in `S3FileIOProperties`, this is generic configuration
for all AWS clients, thus the prefix `client.`. So we could create a class
`AwsClientProperties` for them
--
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]
