jackye1995 commented on code in PR #5902:
URL: https://github.com/apache/iceberg/pull/5902#discussion_r985299777
##########
docs/aws.md:
##########
@@ -573,6 +573,55 @@ spark-sql --packages
org.apache.iceberg:iceberg-spark3-runtime:{{% icebergVersio
--conf
spark.sql.catalog.my_catalog.client.assume-role.region=ap-northeast-1
```
+### URL Connection HTTP Client
Review Comment:
can we make this level 4, and at level 3 have a dedicated section for `HTTP
client configurations`?
##########
docs/aws.md:
##########
@@ -573,6 +573,55 @@ spark-sql --packages
org.apache.iceberg:iceberg-spark3-runtime:{{% icebergVersio
--conf
spark.sql.catalog.my_catalog.client.assume-role.region=ap-northeast-1
```
+### URL Connection HTTP Client
Review Comment:
And although it is just 1 line at this moment, can we also have a table for
choosing the client type under the level 3 section? And we can talk a bit about
the pros and cons of different HTTP client implementations
--
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]
