gianm commented on a change in pull request #6953: Move GCP to a core extension
URL: https://github.com/apache/incubator-druid/pull/6953#discussion_r263153644
##########
File path: server/pom.xml
##########
@@ -42,6 +42,11 @@
<artifactId>druid-aws-common</artifactId>
<version>${project.parent.version}</version>
</dependency>
+ <dependency>
+ <groupId>org.apache.druid</groupId>
+ <artifactId>druid-gcp-common</artifactId>
Review comment:
Would it work to add `aws-common` and `gcp-common` as dependencies for each
extension individually, not as their own extensions? So we aren't doing
extension-depends-on-extension, but are instead loading the aws/gcp common
jars, separately, into each extension classloader?
----------------------------------------------------------------
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]
