yuqi1129 commented on code in PR #4250: URL: https://github.com/apache/gravitino/pull/4250#discussion_r1690719413
########## docs/iceberg-rest-service.md: ########## @@ -163,7 +179,7 @@ For example, we can configure Spark catalog options to use Gravitino Iceberg RES --conf spark.sql.catalog.rest.uri=http://127.0.0.1:9001/iceberg/ ``` -You may need to adjust the Iceberg Spark runtime jar file name according to the real version number in your environment. +You may need to adjust the Iceberg Spark runtime jar file name according to the real version number in your environment. If accessing the data stored in S3, you should download [iceberg-aws-bundle](https://mvnrepository.com/artifact/org.apache.iceberg/iceberg-aws-bundle) and set catalog properties `s3.access-key-id`, `s3.secret-access-key` explicitly. Review Comment: If accessing -> If you want to access... ########## docs/iceberg-rest-service.md: ########## @@ -119,6 +119,22 @@ The `clients` property for example: `catalog-impl` has no effect. ::: +### S3 configuration + +Gravitino Iceberg REST service support using static access-key-id and secret-access-key to access S3 data. Review Comment: What's the meaning of word `static` here? -- 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]
