[GitHub] [druid] maytasm3 commented on issue #9305: Add support for optional cloud (aws, gcs, etc.) credentials for s3 for ingestion

[GitBox <gi...@apache.org>]

maytasm3 commented on issue #9305: Add support for optional cloud (aws, gcs, etc.) credentials for s3 for ingestion
URL: https://github.com/apache/druid/issues/9305#issuecomment-583189773
 
 
   @zachjsh 
   - The client instantiation is done inside the peon process which is forked from the middleManager. It might be difficult to keep the old client around and reuse it. We can do that as a further optimization if required. Are you worried about the cost for initializing the client? 
   - We have PasswordProvider which abstract out the underlying mechanism to retrieve the password. There is additional work to implement file path password and env variable as key for encrypt/decrypt password.       
   - This solution does not support multiple cloud storage providers at one time. Supporting multiple cloud storage providers in single ingestion can be done using the the "Multi" input source idea. Out of scope for this proposal 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org