snazy commented on code in PR #3591:
URL: https://github.com/apache/polaris/pull/3591#discussion_r2740704449
##########
getting-started/ozone/README.md:
##########
@@ -48,6 +48,7 @@ bin/spark-sql \
--conf spark.sql.catalog.polaris.token-refresh-enabled=false \
--conf spark.sql.catalog.polaris.warehouse=quickstart_catalog \
--conf spark.sql.catalog.polaris.scope=PRINCIPAL_ROLE:ALL \
+ --conf spark.sql.catalog.polaris.header.X-Iceberg-Access-Delegation="" \
Review Comment:
I'd prefer to have the guides-testing in, hence this PR.
Manual testing (copy paste from the Markdown) has different (timing)
behavior than running the steps "as fast as possible" from the CI scripts.
The `Credential vending was requested for table [...], but no credentials
are available` message is coming from Polaris, so I think this is an issue in
the code base as it exists `main`.
Related/orthogonal: I'm not sure that using static credentials passed to a
client (Spark SQL shell in this case) is a good option to demonstrate to users.
What's missing here is S3 remote request signing.
--
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]
