FANNG1 commented on code in PR #4250: URL: https://github.com/apache/gravitino/pull/4250#discussion_r1699535204
########## conf/gravitino-iceberg-rest-server.conf.template: ########## @@ -41,7 +41,23 @@ gravitino.iceberg-rest.requestHeaderSize = 131072 gravitino.iceberg-rest.responseHeaderSize = 131072 # THE CONFIGURATION FOR Iceberg catalog backend -# The Iceberg catalog backend, it's recommended to change to hive or jdbc +# The Iceberg catalog backend, it's recommanded to change to hive or jdbc Review Comment: done ########## docs/iceberg-rest-service.md: ########## @@ -163,7 +181,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 you want to access the data stored in S3, you should download [iceberg-aws-bundle](https://mvnrepository.com/artifact/org.apache.iceberg/iceberg-aws-bundle) and no extra config is needed because S3 related properties is transferred from Iceberg REST server to Iceberg REST client. Review Comment: updated -- 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]
