Skip to content

exclude avro extension dependencies that are already included in druid core libraries#8309

Merged
jon-wei merged 1 commit intoapache:masterfrom
clintropolis:chillax-avro-dependencies
Aug 16, 2019
Merged

exclude avro extension dependencies that are already included in druid core libraries#8309
jon-wei merged 1 commit intoapache:masterfrom
clintropolis:chillax-avro-dependencies

Conversation

@clintropolis
Copy link
Copy Markdown
Member

Description

While working on NOTICE stuff for 0.16.0, noticed avro extension had a lot of dependencies that were already included in lib/ folder of package, so I have excluded them from the avro extension pom.

before:
Screen Shot 2019-08-14 at 8 09 50 PM

$ ls -1
avro-1.9.0.jar
avro-ipc-1.9.0.jar
avro-ipc-jetty-1.9.0.jar
avro-mapred-1.9.0.jar
commons-compress-1.18.jar
commons-lang3-3.5.jar
druid-avro-extensions-0.16.0-incubating-SNAPSHOT.jar
gson-2.3.1.jar
javax.annotation-api-1.3.2.jar
javax.servlet-api-3.1.0.jar
jersey-client-1.15.jar
jersey-core-1.19.3.jar
jetty-http-9.4.10.v20180503.jar
jetty-io-9.4.10.v20180503.jar
jetty-security-9.4.10.v20180503.jar
jetty-server-9.4.10.v20180503.jar
jetty-servlet-9.4.10.v20180503.jar
jetty-util-9.4.10.v20180503.jar
jsr311-api-1.1.1.jar
kafka-schema-registry-client-3.0.1.jar
schema-repo-api-0.1.3.jar
schema-repo-avro-0.1.3.jar
schema-repo-client-0.1.3.jar
schema-repo-common-0.1.3.jar
velocity-engine-core-2.0.jar

after:
Screen Shot 2019-08-14 at 8 09 00 PM

$ ls -1
avro-1.9.0.jar
avro-ipc-1.9.0.jar
avro-ipc-jetty-1.9.0.jar
avro-mapred-1.9.0.jar
druid-avro-extensions-0.16.0-incubating-SNAPSHOT.jar
gson-2.3.1.jar
javax.annotation-api-1.3.2.jar
jersey-client-1.15.jar
kafka-schema-registry-client-3.0.1.jar
schema-repo-api-0.1.3.jar
schema-repo-avro-0.1.3.jar
schema-repo-client-0.1.3.jar
schema-repo-common-0.1.3.jar
velocity-engine-core-2.0.jar

@clintropolis clintropolis added WIP and removed WIP labels Aug 16, 2019
@clintropolis
Copy link
Copy Markdown
Member Author

tested indexing avro file in hadoop, succeeded

@jon-wei jon-wei merged commit 7362b1d into apache:master Aug 16, 2019
@clintropolis clintropolis deleted the chillax-avro-dependencies branch August 16, 2019 22:46
jihoonson pushed a commit to implydata/druid-public that referenced this pull request Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants