drcrallen opened a new issue #6910: Bring GCP functionality to core 
functionality
URL: https://github.com/apache/incubator-druid/issues/6910
 
 
   We heavily use the Google Cloud extensions internally, and have encountered 
some pain points with it being in a contrib extension.
   
   Like the AWS module at 
https://github.com/apache/incubator-druid/tree/master/aws-common this ask is 
that there be a GCP common which supplies a default 
`com.google.api.client.http.HttpRequestInitializer`, 
`com.google.api.client.http.HttpTransport`, and 
`com.google.api.client.json.JsonFactory`. Any extensions which use the client 
api can then pull from the parent classpath's instance of these or supply their 
own. And modules are able to then override the default supplied ones if 
required.
   
   The overall suggestion is to do the following (in no specific order):
   
   1. Move `aws-common` into `cloud/aws-common`
   2. Create `gcp-common` at `cloud/gcp-common` which contains a simple module 
for supplying `com.google.api.client.http.HttpRequestInitializer`, 
`com.google.api.client.http.HttpTransport`, and 
`com.google.api.client.json.JsonFactory`
   3. Move the google-extension into extensions-core , we are willing to put 
effort into maintaining this extension as time goes on.
   4. Modify google-extension to use the base gcp-common.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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]

Reply via email to