sthetland commented on a change in pull request #9375: Add support for optional cloud (aws, gcs, etc.) credentials for s3 for ingestion URL: https://github.com/apache/druid/pull/9375#discussion_r383530823
########## File path: docs/development/extensions-core/s3.md ########## @@ -64,6 +64,7 @@ In addition to this you need to set additional configuration, specific for [deep ### S3 authentication methods To connect to your S3 bucket (whether deep storage bucket or source bucket), Druid use the following credentials providers chain +**Note :** *Below authentication chain can be overridden for connecting to source bucket by specifying the [properties](../../ingestion/native-batch.md#s3-input-source) parameter in the ingestionSpec.* Review comment: I think the doc normally uses a carrot to format note-style paragraphs, like the ones on https://druid.apache.org/docs/latest/tutorials/cluster.html#with-zookeeper-on-master for example. I'd use that style, or this could just be a regular second sentence in the paragraph. In either case, I'd rephrase a bit: > You can override the default credentials provider chain by specifying an access key using [Properties Object parameters](../../ingestion/native-batch.md#s3-input-source) in the ingestionSpec. ---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
