amaechler commented on code in PR #20540: URL: https://github.com/apache/druid/pull/20540#discussion_r4338818249
########## docs/multi-stage-query/reference.md: ########## @@ -152,7 +152,7 @@ Supported arguments for the function: |---|---|---|---| | `bucket` | Yes | S3 bucket destination for exported files. You must add the bucket and prefix combination to the `druid.export.storage.s3.allowedExportPaths` allow list. | n/a | | `prefix` | Yes | Destination path in the bucket to create exported files. The export query expects the destination path to be empty. If the location includes other files, the query will fail. You must add the bucket and prefix combination to the `druid.export.storage.s3.allowedExportPaths` allow list. | n/a | -| `assumeRoleArn` | No | ARN of the role to assume before exporting data. If not provided, the default credentials configured for the Druid process are used. | n/a | +| `assumeRoleArn` | No | ARN of the role to assume before exporting data. If not provided, the default credentials configured for the Druid process are used. If the Druid process sets `druid.s3.intermediateAssumeRoleArn`, those credentials assume that role first. | n/a | Review Comment: The intermediate role only applies when `assumeRoleArn` is set, so this row could say so. ```suggestion | `assumeRoleArn` | No | ARN of the role to assume before exporting data. If not provided, the default credentials configured for the Druid process are used. If provided and the Druid process sets `druid.s3.intermediateAssumeRoleArn`, those credentials assume that role before assuming `assumeRoleArn`. | n/a | ``` -- 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]
