MartijnVisser edited a comment on pull request #17635: URL: https://github.com/apache/flink/pull/17635#issuecomment-960686850
@fapaul `flink-connector-hbase-1.4` pulls `jaxb-impl` in via Hadoop dependencies, which is the same behaviour as currently on master. ``` [INFO] org.apache.flink:flink-connector-hbase-1.4:jar:1.15-SNAPSHOT [INFO] +- org.apache.hbase:hbase-server:test-jar:tests:1.4.3:test [INFO] | +- org.apache.hbase:hbase-common:jar:1.4.3:compile [INFO] | +- org.apache.hbase:hbase-protocol:jar:1.4.3:compile [INFO] | +- org.apache.hbase:hbase-procedure:jar:1.4.3:test [INFO] | +- org.apache.hbase:hbase-prefix-tree:jar:1.4.3:test [INFO] | +- org.apache.hbase:hbase-metrics-api:jar:1.4.3:test [INFO] | +- org.apache.hbase:hbase-metrics:jar:1.4.3:test [INFO] | | \- io.dropwizard.metrics:metrics-core:jar:3.1.2:test [INFO] | +- commons-httpclient:commons-httpclient:jar:3.1:compile [INFO] | +- commons-codec:commons-codec:jar:1.15:compile [INFO] | +- org.apache.hbase:hbase-hadoop-compat:jar:1.4.3:test [INFO] | +- org.apache.hbase:hbase-hadoop2-compat:jar:1.4.3:test [INFO] | +- com.yammer.metrics:metrics-core:jar:2.2.0:compile [INFO] | +- com.google.protobuf:protobuf-java:jar:2.5.0:compile [INFO] | +- com.sun.jersey:jersey-core:jar:1.9:compile [INFO] | +- com.sun.jersey:jersey-server:jar:1.9:compile [INFO] | | \- asm:asm:jar:3.1:compile [INFO] | +- commons-cli:commons-cli:jar:1.4:compile [INFO] | +- commons-io:commons-io:jar:2.11.0:compile [INFO] | +- commons-lang:commons-lang:jar:2.6:compile [INFO] | +- commons-logging:commons-logging:jar:1.1.3:compile [INFO] | +- org.apache.commons:commons-math:jar:2.2:test [INFO] | +- org.apache.zookeeper:zookeeper:jar:3.4.14:compile [INFO] | | +- com.github.spotbugs:spotbugs-annotations:jar:3.1.9:compile [INFO] | | \- org.apache.yetus:audience-annotations:jar:0.5.0:compile [INFO] | +- org.mortbay.jetty:jetty:jar:6.1.26:compile [INFO] | +- org.mortbay.jetty:jetty-util:jar:6.1.26:compile [INFO] | +- org.mortbay.jetty:jetty-sslengine:jar:6.1.26:test [INFO] | +- org.mortbay.jetty:jsp-2.1:jar:6.1.14:test [INFO] | +- org.mortbay.jetty:jsp-api-2.1:jar:6.1.14:test [INFO] | +- org.mortbay.jetty:servlet-api-2.5:jar:6.1.14:test [INFO] | +- org.codehaus.jackson:jackson-core-asl:jar:1.9.13:compile [INFO] | +- org.codehaus.jackson:jackson-mapper-asl:jar:1.9.13:compile [INFO] | +- org.codehaus.jackson:jackson-jaxrs:jar:1.9.13:compile [INFO] | +- tomcat:jasper-compiler:jar:5.5.23:runtime [INFO] | +- tomcat:jasper-runtime:jar:5.5.23:runtime [INFO] | | \- commons-el:commons-el:jar:1.0:runtime [INFO] | +- org.jamon:jamon-runtime:jar:2.4.1:test [INFO] | +- org.apache.htrace:htrace-core:jar:3.1.0-incubating:compile [INFO] | +- com.lmax:disruptor:jar:3.4.2:test [INFO] | +- org.apache.httpcomponents:httpclient:jar:4.5.13:compile [INFO] | +- org.apache.httpcomponents:httpcore:jar:4.4.14:compile [INFO] | +- org.apache.hadoop:hadoop-common:jar:2.4.1:compile [INFO] | | +- org.apache.hadoop:hadoop-annotations:jar:2.4.1:compile [INFO] | | +- commons-net:commons-net:jar:3.1:compile [INFO] | | +- com.sun.jersey:jersey-json:jar:1.9:compile [INFO] | | | +- org.codehaus.jettison:jettison:jar:1.1:compile [INFO] | | | +- com.sun.xml.bind:jaxb-impl:jar:2.2.3-1:compile ``` Same thing for `flink-connector-hbase-2.2` ``` [INFO] org.apache.flink:flink-connector-hbase-2.2:jar:1.15-SNAPSHOT [INFO] +- org.apache.hadoop:hadoop-common:jar:2.4.1:provided [INFO] | +- org.apache.hadoop:hadoop-annotations:jar:2.4.1:provided [INFO] | +- com.google.guava:guava:jar:11.0.2:provided [INFO] | +- commons-cli:commons-cli:jar:1.4:provided [INFO] | +- xmlenc:xmlenc:jar:0.52:provided [INFO] | +- commons-httpclient:commons-httpclient:jar:3.1:provided [INFO] | +- commons-codec:commons-codec:jar:1.15:compile [INFO] | +- commons-io:commons-io:jar:2.11.0:compile [INFO] | +- commons-net:commons-net:jar:3.1:provided [INFO] | +- javax.servlet:servlet-api:jar:2.5:provided [INFO] | +- org.mortbay.jetty:jetty:jar:6.1.26:provided [INFO] | +- org.mortbay.jetty:jetty-util:jar:6.1.26:provided [INFO] | +- com.sun.jersey:jersey-core:jar:1.9:provided [INFO] | +- com.sun.jersey:jersey-json:jar:1.9:provided [INFO] | | +- org.codehaus.jettison:jettison:jar:1.1:provided [INFO] | | +- com.sun.xml.bind:jaxb-impl:jar:2.2.3-1:provided ``` **NB: I removed not applicable lines for readability purposes in both overviews** What is interesting, is that both `flink-sql-connector-hbase-1.4` and `flink-sql-connector-hbase-2.2` don't have this dependency and have an overall smaller dependency footprint. I think we should open a follow-up Jira for having one HBase connector, which should have that small dependency footprint. LMKWYT -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
