xingbowu commented on a change in pull request #3553:
URL: https://github.com/apache/iceberg/pull/3553#discussion_r752804444



##########
File path: aliyun/src/main/java/org/apache/iceberg/aliyun/AliyunProperties.java
##########
@@ -25,21 +25,76 @@
 import org.apache.iceberg.util.PropertyUtil;
 
 public class AliyunProperties implements Serializable {
+  /**
+   * The domain name used to access OSS. OSS uses HTTP Restful APIs to provide 
services. Different regions are accessed
+   * by using different endpoints. For the same region, access over the 
internal network or over the Internet also uses
+   * different endpoints. For more information, see:
+   * https://www.alibabacloud.com/help/doc-detail/31837.htm
+   */
+  public static final String OSS_ENDPOINT = "oss.endpoint";
+
+  /**
+   * Aliyun uses an AccessKey pair, which includes an AccessKey ID and an 
AccessKey secret to implement symmetric
+   * encryption and verify the identity of a requester. The AccessKey ID is 
used to identify a user.
+   * <p>
+   * For more information about how to obtain an AccessKey pair, see:
+   * https://www.alibabacloud.com/help/doc-detail/53045.htm
+   */
+  public static final String ACCESS_KEY_ID = "access.key.id";
+
+  /**
+   * Aliyun uses an AccessKey pair, which includes an AccessKey ID and an 
AccessKey secret to implement symmetric
+   * encryption and verify the identity of a requester.  The AccessKey secret 
is used to encrypt and verify the
+   * signature string.
+   * <p>
+   * For more information about how to obtain an AccessKey pair, see:
+   * https://www.alibabacloud.com/help/doc-detail/53045.htm
+   */
+  public static final String ACCESS_KEY_SECRET = "access.key.secret";
+
+  /**
+   * The implementation class of {@link AliyunClientFactory} to customize 
Aliyun client configurations.
+   * If set, all Aliyun clients will be initialized by the specified factory.
+   * If not set, {@link DefaultAliyunClientFactory} is used as default factory.
+   */
+  public static final String CLIENT_FACTORY = "client.factory-impl";

Review comment:
       I can make another PR to line up the behavior of aws part as 
"client.factory-impl" 




-- 
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]

Reply via email to