himanshug commented on issue #9518: AWS RDS token based password provider URL: https://github.com/apache/druid/pull/9518#issuecomment-604272488 removing `Area - SQL` makes sense now that I think about it. I actually thought of putting the new `PasswordProvider` in an extension. Looked at existing aws related extensions and it did not fit in any of those , the `ec2-extensions` or `s3-extensions` . So, had put there along side other common AWS related code. Anyways, It is totally possible to put `AWSRDSTokenPasswordProvider` in a new extension (`aws-rds-extensions`). However, `BasicDataSourceExt` would stay in core. I will update the PR . thanks.
---------------------------------------------------------------- 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]
