sunxiaojian commented on code in PR #5701:
URL: https://github.com/apache/gravitino/pull/5701#discussion_r1870523543


##########
docs/iceberg-rest-service.md:
##########
@@ -128,16 +128,20 @@ To configure the JDBC catalog backend, set the 
`gravitino.iceberg-rest.warehouse
 
 #### OSS configuration
 
-Gravitino Iceberg REST service supports using static access-key-id and 
secret-access-key to access OSS data.
+Gravitino Iceberg REST service supports using static access-key-id and 
secret-access-key or generating temporary token to access OSS data.
 
-| Configuration item                             | Description                 
                                                                          | 
Default value | Required | Since Version    |
-|------------------------------------------------|-------------------------------------------------------------------------------------------------------|---------------|----------|------------------|
-| `gravitino.iceberg-rest.io-impl`               | The IO implementation for 
`FileIO` in Iceberg, use `org.apache.iceberg.aliyun.oss.OSSFileIO` for OSS. | 
(none)        | No       | 0.6.0-incubating |
-| `gravitino.iceberg-rest.oss-access-key-id`     | The static access key ID 
used to access OSS data.                                                     | 
(none)        | No       | 0.7.0-incubating |
-| `gravitino.iceberg-rest.oss-secret-access-key` | The static secret access 
key used to access OSS data.                                                 | 
(none)        | No       | 0.7.0-incubating |
-| `gravitino.iceberg-rest.oss-endpoint`          | The endpoint of Aliyun OSS 
service.                                                                   | 
(none)        | No       | 0.7.0-incubating |
+| Configuration item                                | Description              
                                                                                
        | Default value   | Required | Since Version    |
+|---------------------------------------------------|------------------------------------------------------------------------------------------------------------------|-----------------|----------|------------------|
+| `gravitino.iceberg-rest.io-impl`                  | The IO implementation 
for `FileIO` in Iceberg, use `org.apache.iceberg.aliyun.oss.OSSFileIO` for OSS. 
           | (none)          | No       | 0.6.0-incubating |
+| `gravitino.iceberg-rest.oss-access-key-id`        | The static access key ID 
used to access OSS data.                                                        
        | (none)          | No       | 0.7.0-incubating |
+| `gravitino.iceberg-rest.oss-secret-access-key`    | The static secret access 
key used to access OSS data.                                                    
        | (none)          | No       | 0.7.0-incubating |
+| `gravitino.iceberg-rest.oss-endpoint`             | The endpoint of Aliyun 
OSS service.                                                                    
          | (none)          | No       | 0.7.0-incubating |
+| `gravitino.iceberg-rest.oss-region`               | The region of the OSS 
service, like `oss-cn-hangzhou`, only used when `credential-provider-type` is 
`oss-token`. | (none)          | No       | 0.8.0-incubating |
+| `gravitino.iceberg-rest.oss-role-arn`             | The ARN of the role to 
access the OSS data, only used when `credential-provider-type` is `oss-token`.  
          | (none)          | No       | 0.8.0-incubating |
+| `gravitino.iceberg-rest.oss-external-id`          | The OSS external id to 
generate token, only used when `credential-provider-type` is `oss-token`.       
          | (none)          | No       | 0.8.0-incubating |
+| `gravitino.iceberg-rest.oss-token-expire-in-secs` | The OSS security token 
expire time in secs,  only used when `credential-provider-type` is `oss-token`. 
          | 3600            | No       | 0.8.0-incubating |
 
-For other Iceberg OSS properties not managed by Gravitino like 
`client.security-token`, you could config it directly by 
`gravitino.iceberg-rest.client.security-token`.

Review Comment:
   reverted



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

Reply via email to