massdosage commented on a change in pull request #2072:
URL: https://github.com/apache/iceberg/pull/2072#discussion_r557494239



##########
File path: site/docs/aws.md
##########
@@ -22,24 +22,34 @@ This section describes how to use Iceberg with AWS.
 
 ## Enabling AWS Integration
 
-The `iceberg-aws` module is bundled with Spark and Flink engine runtimes.
+The `iceberg-aws` module is bundled with Spark and Flink engine runtimes for 
all versions starting `0.11.0`.

Review comment:
       ```suggestion
   The `iceberg-aws` module is bundled with Spark and Flink engine runtimes for 
all versions from `0.11.0` onwards.
   ```

##########
File path: site/docs/aws.md
##########
@@ -22,24 +22,34 @@ This section describes how to use Iceberg with AWS.
 
 ## Enabling AWS Integration
 
-The `iceberg-aws` module is bundled with Spark and Flink engine runtimes.
+The `iceberg-aws` module is bundled with Spark and Flink engine runtimes for 
all versions starting `0.11.0`.
 However, the AWS clients are not bundled so that you can use the same client 
version as your application.
 You will need to provide the AWS v2 SDK because that is what Iceberg depends 
on.
 You can choose to use the [AWS SDK 
bundle](https://mvnrepository.com/artifact/software.amazon.awssdk/bundle), 
 or individual AWS client packages (Glue, S3, DynamoDB, KMS, STS) if you would 
like to have a minimal dependency footprint.
 
+All the default AWS clients use the [URL Connection HTTP 
Client](https://mvnrepository.com/artifact/software.amazon.awssdk/url-connection-client)
+for HTTP connection management.
+This dependency is not a part of AWS SDK bundle and needs to be added 
separately.

Review comment:
       ```suggestion
   This dependency is not part of the AWS SDK bundle and needs to be added 
separately.
   ```




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

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