o-nikolas commented on code in PR #76086:
URL: https://github.com/apache/airflow/pull/76086#discussion_r4429172418
##########
providers/amazon/docs/operators/duckdb.rst:
##########
@@ -132,9 +132,10 @@ DuckDB how to authenticate, and the
``credential_strategy`` parameter selects ho
``credential_chain`` (default)
Issues ``CREATE SECRET (TYPE s3, PROVIDER credential_chain)``, which
resolves credentials through
- the AWS SDK inside DuckDB's ``aws`` extension. Nothing secret is written
into SQL text and the SDK
- refreshes expiring credentials itself, so a long query does not fail
partway through when a set of
- temporary credentials expires.
+ the AWS SDK inside DuckDB's ``aws`` extension. Nothing secret is written
into SQL text, and the
+ secret is created with ``REFRESH 'auto'``: when S3 rejects expired
credentials, DuckDB resolves the
+ chain again and retries, so a long query does not fail partway through
when a set of temporary
+ credentials expires.
Review Comment:
Yeah, fair callout, I'll update that, it will only mention the 1.2.2 to
1.5.5 since we'll make 1.2.2 the floor, per your other comment.
--
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]
