You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "cyrilou242 (via GitHub)" <gi...@apache.org> on 2023/09/04 13:02:24 UTC
[GitHub] [pinot] cyrilou242 opened a new issue, #11507: pinot-clients have too many dependencies
cyrilou242 opened a new issue, #11507:
URL: https://github.com/apache/pinot/issues/11507
## Problem
Since 0.11.0, `pinot-jdbc-client` and pinot `pinot-java-client` have a dependency on `pinot-common`.
See introductions:
https://github.com/apache/pinot/commit/15e939818ca913bd9a5f0162300057c1a8b20f39
https://github.com/apache/pinot/commit/23a81d07b52dad6181b696562e07cdfa0932d191
This introduces the following new dependencies to the clients:
```
\- org.apache.pinot:pinot-common:jar:0.12.1:compile
+- org.apache.httpcomponents:httpmime:jar:4.5.13:compile
+- org.antlr:antlr4-runtime:jar:4.6:compile
+- org.apache.thrift:libthrift:jar:0.15.0:compile
| \- javax.annotation:javax.annotation-api:jar:1.3.2:compile
+- org.xerial.snappy:snappy-java:jar:1.1.8.2:compile
+- com.github.luben:zstd-jni:jar:1.5.2-3:compile
+- org.lz4:lz4-java:jar:1.8.0:compile
+- org.apache.logging.log4j:log4j-slf4j-impl:jar:2.17.1:compile
| \- org.apache.logging.log4j:log4j-core:jar:2.17.1:runtime
+- commons-httpclient:commons-httpclient:jar:3.1:compile
+- it.unimi.dsi:fastutil:jar:8.2.3:compile
+- org.webjars:swagger-ui:jar:3.23.11:compile
+- io.grpc:grpc-netty-shaded:jar:1.41.0:compile
| +- io.perfmark:perfmark-api:jar:0.23.0:runtime
| \- io.grpc:grpc-core:jar:1.41.0:compile
| +- com.google.code.gson:gson:jar:2.8.6:runtime
| +- com.google.android:annotations:jar:4.1.1.4:runtime
| \- org.codehaus.mojo:animal-sniffer-annotations:jar:1.19:runtime
+- io.grpc:grpc-protobuf:jar:1.41.0:compile
| +- io.grpc:grpc-api:jar:1.41.0:compile
| | \- io.grpc:grpc-context:jar:1.41.0:compile
| +- com.google.api.grpc:proto-google-common-protos:jar:2.0.1:compile
| \- io.grpc:grpc-protobuf-lite:jar:1.41.0:compile
+- io.grpc:grpc-stub:jar:1.41.0:compile
+- org.apache.yetus:audience-annotations:jar:0.13.0:compile
+- org.mindrot:jbcrypt:jar:0.4:compile
\- com.github.seancfoley:ipaddress:jar:5.3.4:compile
```
This is a lot of new, unused dependencies for the client.
Also notice that an implementation of log4j is included `org.apache.logging.log4j:log4j-slf4j-impl:jar:2.17.1:compile`, which is bad practice but easy to quickfix with an exclusion in the pom.
Then for `pinot-jdbc-client` a dependency on `pinot-core` was introduced:
https://github.com/apache/pinot/commit/caf8d755820d6bf27bc7daeb74ac35d2e70caa61
This introduces the following dependencies in the client
```
+- org.apache.pinot:pinot-jdbc-client:jar:0.12.1:compile
| +- org.apache.pinot:pinot-core:jar:0.12.1:compile
| | +- com.uber:h3:jar:4.0.0:compile
| | +- org.roaringbitmap:RoaringBitmap:jar:0.9.35:compile
| | | \- org.roaringbitmap:shims:jar:0.9.35:runtime
| | +- org.apache.pinot:pinot-segment-spi:jar:0.12.1:compile
| | +- org.apache.pinot:pinot-segment-local:jar:0.12.1:compile
| | +- io.netty:netty-transport-native-epoll:jar:linux-x86_64:4.1.79.Final:compile
| | | +- io.netty:netty-common:jar:4.1.79.Final:compile
| | | +- io.netty:netty-buffer:jar:4.1.79.Final:compile
| | | +- io.netty:netty-transport:jar:4.1.79.Final:compile
| | | +- io.netty:netty-transport-native-unix-common:jar:4.1.79.Final:compile
| | | \- io.netty:netty-transport-classes-epoll:jar:4.1.79.Final:compile
| | +- io.netty:netty-transport-native-kqueue:jar:osx-x86_64:4.1.79.Final:compile
| | | \- io.netty:netty-transport-classes-kqueue:jar:4.1.79.Final:compile
| | +- io.netty:netty-tcnative-boringssl-static:jar:linux-x86_64:2.0.53.Final:compile
| | | +- io.netty:netty-tcnative-classes:jar:2.0.53.Final:compile
| | | +- io.netty:netty-tcnative-boringssl-static:jar:linux-aarch_64:2.0.53.Final:compile
| | | +- io.netty:netty-tcnative-boringssl-static:jar:osx-aarch_64:2.0.53.Final:compile
| | | \- io.netty:netty-tcnative-boringssl-static:jar:windows-x86_64:2.0.53.Final:compile
| | +- io.netty:netty-tcnative-boringssl-static:jar:osx-x86_64:2.0.53.Final:compile
| | +- io.netty:netty-all:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-dns:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-haproxy:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-http2:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-memcache:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-mqtt:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-redis:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-smtp:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-stomp:jar:4.1.79.Final:compile
| | | +- io.netty:netty-codec-xml:jar:4.1.79.Final:compile
| | | +- io.netty:netty-resolver:jar:4.1.79.Final:compile
| | | +- io.netty:netty-resolver-dns:jar:4.1.79.Final:compile
| | | +- io.netty:netty-transport-rxtx:jar:4.1.79.Final:compile
| | | +- io.netty:netty-transport-sctp:jar:4.1.79.Final:compile
| | | +- io.netty:netty-transport-udt:jar:4.1.79.Final:compile
| | | +- io.netty:netty-resolver-dns-classes-macos:jar:4.1.79.Final:compile
| | | +- io.netty:netty-transport-native-epoll:jar:linux-aarch_64:4.1.79.Final:runtime
| | | +- io.netty:netty-transport-native-kqueue:jar:osx-aarch_64:4.1.79.Final:runtime
| | | +- io.netty:netty-resolver-dns-native-macos:jar:osx-x86_64:4.1.79.Final:runtime
| | | \- io.netty:netty-resolver-dns-native-macos:jar:osx-aarch_64:4.1.79.Final:runtime
| | +- com.clearspring.analytics:stream:jar:2.7.0:compile
| | +- org.apache.datasketches:datasketches-java:jar:1.2.0-incubating:compile
| | | \- org.apache.datasketches:datasketches-memory:jar:1.2.0-incubating:compile
| | +- com.tdunning:t-digest:jar:3.2:compile
| | +- org.xerial.larray:larray-mmap:jar:0.4.1:compile
| | | \- org.xerial.larray:larray-buffer:jar:0.4.1:compile
| | +- net.sf.jopt-simple:jopt-simple:jar:4.6:compile
| | +- org.glassfish.jersey.containers:jersey-container-grizzly2-http:jar:2.35:compile
| | | \- org.glassfish.hk2.external:jakarta.inject:jar:2.6.1:compile
| | +- org.glassfish.grizzly:grizzly-http-server:jar:2.4.4:compile
| | +- org.glassfish.hk2:hk2-locator:jar:2.6.1:compile
| | +- org.apache.lucene:lucene-core:jar:8.2.0:compile
| | +- org.apache.lucene:lucene-queryparser:jar:8.2.0:compile
| | | +- org.apache.lucene:lucene-queries:jar:8.2.0:compile
| | | \- org.apache.lucene:lucene-sandbox:jar:8.2.0:compile
| | \- org.apache.lucene:lucene-analyzers-common:jar:8.2.0:compile
| +- org.asynchttpclient:async-http-client:jar:2.12.3:compile
| | +- org.asynchttpclient:async-http-client-netty-utils:jar:2.12.3:compile
| | +- io.netty:netty-codec-http:jar:4.1.60.Final:compile
| | +- io.netty:netty-handler:jar:4.1.60.Final:compile
| | +- io.netty:netty-codec-socks:jar:4.1.60.Final:compile
| | +- io.netty:netty-handler-proxy:jar:4.1.60.Final:compile
| | +- org.reactivestreams:reactive-streams:jar:1.0.3:compile
| | +- com.typesafe.netty:netty-reactive-streams:jar:2.0.4:compile
| | \- com.sun.activation:jakarta.activation:jar:1.2.2:compile
| \- com.101tec:zkclient:jar:0.7:compile
```
Impact:
The pinot clients are too heavy and introduce too many unused dependencies that can conflict with the users's project. We are stuck on client `0.10.0`.
## Suggestion
From what I understand of the commits the goal of importing pinot-common and pinot-core was to get access to a few utils, and almost all of the dependencies are not used.
I'd be nice to refactor dependencies to make the pinot clients light again.
--
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: commits-unsubscribe@pinot.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] xiangfu0 commented on issue #11507: pinot-clients have too many dependencies
Posted by "xiangfu0 (via GitHub)" <gi...@apache.org>.
xiangfu0 commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-1708392274
Another approach is to remove the dependencies of `pinot-common`, only `pinot-spi`.
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] xiangfu0 commented on issue #11507: pinot-clients have too many dependencies
Posted by "xiangfu0 (via GitHub)" <gi...@apache.org>.
xiangfu0 commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-1722285263
@cyrilou242 I think for now you may need to explicitly exclude libs from `pinot-jdbc-client`
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] pinot-clients have too many dependencies [pinot]
Posted by "cyrilou242 (via GitHub)" <gi...@apache.org>.
cyrilou242 commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-2486104914
Update on `pinot-java-client` dependencies for reference, could be helpful to other users.
`pinot-java-client 0.10.0`:
```
+- org.apache.pinot:pinot-java-client:jar:0.10.0:compile
| +- org.asynchttpclient:async-http-client:jar:2.12.3:compile
| | +- org.asynchttpclient:async-http-client-netty-utils:jar:2.12.3:compile
| | | \- io.netty:netty-buffer:jar:4.1.60.Final:compile
| | +- io.netty:netty-codec-http:jar:4.1.60.Final:compile
| | | +- io.netty:netty-common:jar:4.1.60.Final:compile
| | | +- io.netty:netty-transport:jar:4.1.60.Final:compile
| | | \- io.netty:netty-codec:jar:4.1.60.Final:compile
| | +- io.netty:netty-handler:jar:4.1.60.Final:compile
| | | \- io.netty:netty-resolver:jar:4.1.60.Final:compile
| | +- io.netty:netty-codec-socks:jar:4.1.60.Final:compile
| | +- io.netty:netty-handler-proxy:jar:4.1.60.Final:compile
| | +- io.netty:netty-transport-native-epoll:jar:linux-x86_64:4.1.60.Final:compile
| | +- io.netty:netty-transport-native-kqueue:jar:osx-x86_64:4.1.60.Final:compile
| | +- org.reactivestreams:reactive-streams:jar:1.0.3:compile
| | +- com.typesafe.netty:netty-reactive-streams:jar:2.0.4:compile
| | \- com.sun.activation:jakarta.activation:jar:1.2.2:compile
| +- io.netty:netty:jar:3.9.6.Final:compile
| +- com.101tec:zkclient:jar:0.7:compile
| | +- org.slf4j:slf4j-log4j12:jar:1.6.1:compile
| | +- log4j:log4j:jar:1.2.15:compile
| | | \- javax.mail:mail:jar:1.4:compile
| | | \- javax.activation:activation:jar:1.1:compile
| | \- org.apache.zookeeper:zookeeper:jar:3.4.6:compile
| | \- jline:jline:jar:0.9.94:compile
| | \- junit:junit:jar:3.8.1:compile
| \- com.google.code.findbugs:jsr305:jar:3.0.0:compile
```
`pinot-java-client 1.2.0`
```
+- org.apache.pinot:pinot-java-client:jar:1.2.0:compile
| +- org.apache.pinot:pinot-common:jar:1.2.0:compile
| | +- org.apache.pinot:pinot-segment-spi:jar:1.2.0:compile
| | | +- org.roaringbitmap:RoaringBitmap:jar:1.1.0:compile
| | | +- org.locationtech.jts:jts-core:jar:1.19.0:compile
| | | +- org.xerial.larray:larray-mmap:jar:0.4.1:compile
| | | | \- org.xerial.larray:larray-buffer:jar:0.4.1:compile
| | | +- net.openhft:posix:jar:2.26ea1:compile
| | | | +- net.java.dev.jna:jna:jar:5.8.0:compile
| | | | +- net.java.dev.jna:jna-platform:jar:5.8.0:compile
| | | | +- com.github.jnr:jnr-ffi:jar:2.2.15:compile
| | | | | +- com.github.jnr:jffi:jar:1.3.12:compile
| | | | | +- com.github.jnr:jffi:jar:native:1.3.12:runtime
| | | | | +- org.ow2.asm:asm:jar:9.2:compile
| | | | | +- com.github.jnr:jnr-a64asm:jar:1.0.0:compile
| | | | | \- com.github.jnr:jnr-x86asm:jar:1.0.2:compile
| | | | \- com.github.jnr:jnr-constants:jar:0.10.4:compile
| | | \- net.openhft:chronicle-core:jar:2.26ea1:compile
| | | \- net.openhft:chronicle-analytics:jar:2.26ea1:compile
| | +- org.apache.httpcomponents:httpmime:jar:4.5.14:compile
| | +- org.apache.httpcomponents:httpclient:jar:4.5.14:compile
| | +- org.apache.httpcomponents:httpcore:jar:4.4.16:compile
| | +- org.apache.calcite:calcite-core:jar:1.37.0:compile
| | | +- org.apache.calcite:calcite-linq4j:jar:1.37.0:compile
| | | +- org.locationtech.jts.io:jts-io-common:jar:1.19.0:compile
| | | | \- com.googlecode.json-simple:json-simple:jar:1.1.1:compile
| | | +- org.locationtech.proj4j:proj4j:jar:1.2.2:compile
| | | +- org.apache.calcite.avatica:avatica-core:jar:1.25.0:compile
| | | | +- org.apache.httpcomponents.client5:httpclient5:jar:5.2.1:runtime
| | | | | \- org.apache.httpcomponents.core5:httpcore5-h2:jar:5.2:runtime
| | | | \- org.apache.httpcomponents.core5:httpcore5:jar:5.2.3:runtime
| | | +- org.apiguardian:apiguardian-api:jar:1.1.2:compile
| | | +- com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:jar:2.18.1:compile
| | | +- com.google.uzaygezen:uzaygezen-core:jar:0.2:runtime
| | | +- org.apache.commons:commons-text:jar:1.10.0:compile
| | | +- org.codehaus.janino:commons-compiler:jar:3.1.11:runtime
| | | \- org.codehaus.janino:janino:jar:3.1.11:runtime
| | +- org.apache.calcite:calcite-babel:jar:1.37.0:compile
| | +- org.glassfish.jersey.containers:jersey-container-grizzly2-http:jar:2.42:compile
| | | +- org.glassfish.hk2.external:jakarta.inject:jar:2.6.1:compile
| | | +- org.glassfish.grizzly:grizzly-http-server:jar:2.4.4:compile
| | | | \- org.glassfish.grizzly:grizzly-http:jar:2.4.4:compile
| | | | \- org.glassfish.grizzly:grizzly-framework:jar:2.4.4:compile
| | | +- org.glassfish.jersey.core:jersey-common:jar:2.42:compile
| | | | \- org.glassfish.hk2:osgi-resource-locator:jar:1.0.3:compile
| | | \- jakarta.ws.rs:jakarta.ws.rs-api:jar:2.1.6:compile
| | +- org.glassfish.jersey.media:jersey-media-multipart:jar:2.42:compile
| | | \- org.jvnet.mimepull:mimepull:jar:1.9.15:compile
| | +- org.glassfish.jersey.media:jersey-media-json-jackson:jar:2.42:compile
| | | +- org.glassfish.jersey.ext:jersey-entity-filtering:jar:2.42:compile
| | | \- com.fasterxml.jackson.module:jackson-module-jaxb-annotations:jar:2.16.2:compile
| | | \- jakarta.xml.bind:jakarta.xml.bind-api:jar:2.3.3:compile
| | +- org.glassfish.jersey.inject:jersey-hk2:jar:2.42:compile
| | | +- org.glassfish.hk2:hk2-locator:jar:2.6.1:compile
| | | | \- org.glassfish.hk2.external:aopalliance-repackaged:jar:2.6.1:compile
| | | \- org.javassist:javassist:jar:3.30.2-GA:compile
| | +- org.glassfish.hk2:hk2-metadata-generator:jar:2.6.1:compile
| | | +- org.glassfish.hk2:hk2-api:jar:2.6.1:compile
| | | +- org.glassfish.hk2:hk2-utils:jar:2.6.1:compile
| | | \- jakarta.annotation:jakarta.annotation-api:jar:1.3.4:compile
| | +- io.swagger:swagger-jersey2-jaxrs:jar:1.6.14:compile
| | | +- io.swagger:swagger-jaxrs:jar:1.6.14:compile
| | | | \- io.swagger:swagger-core:jar:1.6.14:compile
| | | | +- io.swagger:swagger-models:jar:1.6.14:compile
| | | | | \- io.swagger:swagger-annotations:jar:1.6.14:compile
| | | | \- javax.validation:validation-api:jar:1.1.0.Final:compile
| | | \- org.glassfish.jersey.containers:jersey-container-servlet-core:jar:2.41:compile
| | +- org.webjars:swagger-ui:jar:5.17.14:compile
| | +- jakarta.servlet:jakarta.servlet-api:jar:4.0.4:compile
| | +- org.apache.thrift:libthrift:jar:0.18.1:compile
| | +- org.xerial.snappy:snappy-java:jar:1.1.10.5:compile
| | +- com.github.luben:zstd-jni:jar:1.5.6-3:compile
| | +- org.lz4:lz4-java:jar:1.8.0:compile
| | +- org.apache.logging.log4j:log4j-slf4j2-impl:jar:2.23.1:compile
| | | +- org.apache.logging.log4j:log4j-api:jar:2.23.1:compile
| | | \- org.apache.logging.log4j:log4j-core:jar:2.23.1:compile
| | +- org.apache.logging.log4j:log4j-1.2-api:jar:2.23.1:compile
| | +- com.lmax:disruptor:jar:4.0.0:compile
| | +- org.apache.commons:commons-compress:jar:1.26.2:compile
| | +- org.apache.helix:helix-core:jar:1.3.1:compile
| | | +- org.apache.helix:helix-common:jar:1.3.1:compile
| | | | +- org.apache.helix:metrics-common:jar:1.3.1:compile
| | | | \- org.apache.helix:zookeeper-api:jar:1.3.1:compile
| | | | +- org.apache.helix:metadata-store-directory-common:jar:1.3.1:compile
| | | | \- javax.annotation:javax.annotation-api:jar:1.3.2:compile
| | | +- org.apache.logging.log4j:log4j-slf4j-impl:jar:2.17.1:compile
| | | +- commons-cli:commons-cli:jar:1.2:compile
| | | +- org.apache.commons:commons-math:jar:2.1:compile
| | | +- io.dropwizard.metrics:metrics-core:jar:4.1.14:compile
| | | \- javax.xml.bind:jaxb-api:jar:2.1:compile
| | | +- javax.xml.stream:stax-api:jar:1.0-2:compile
| | | \- javax.activation:activation:jar:1.1:compile
| | +- it.unimi.dsi:fastutil:jar:8.5.13:compile
| | +- net.sf.jopt-simple:jopt-simple:jar:5.0.4:compile
| | +- com.jayway.jsonpath:json-path:jar:2.9.0:compile
| | | \- net.minidev:json-smart:jar:2.5.0:runtime
| | | \- net.minidev:accessors-smart:jar:2.5.0:runtime
| | +- org.slf4j:jcl-over-slf4j:jar:2.0.13:compile
| | +- org.apache.zookeeper:zookeeper:jar:3.9.2:compile
| | | +- org.apache.zookeeper:zookeeper-jute:jar:3.9.2:compile
| | | +- io.netty:netty-tcnative-boringssl-static:jar:2.0.61.Final:compile
| | | | +- io.netty:netty-tcnative-classes:jar:2.0.61.Final:compile
| | | | +- io.netty:netty-tcnative-boringssl-static:jar:linux-x86_64:2.0.61.Final:compile
| | | | +- io.netty:netty-tcnative-boringssl-static:jar:linux-aarch_64:2.0.61.Final:compile
| | | | +- io.netty:netty-tcnative-boringssl-static:jar:osx-x86_64:2.0.61.Final:compile
| | | | +- io.netty:netty-tcnative-boringssl-static:jar:osx-aarch_64:2.0.61.Final:compile
| | | | \- io.netty:netty-tcnative-boringssl-static:jar:windows-x86_64:2.0.61.Final:compile
| | | +- ch.qos.logback:logback-core:jar:1.2.13:compile
| | | \- ch.qos.logback:logback-classic:jar:1.2.13:compile
| | +- org.reflections:reflections:jar:0.10.2:compile
| | +- com.google.protobuf:protobuf-java:jar:3.25.3:compile
| | +- io.grpc:grpc-netty-shaded:jar:1.65.0:compile
| | | +- io.grpc:grpc-util:jar:1.65.0:runtime
| | | | \- org.codehaus.mojo:animal-sniffer-annotations:jar:1.23:runtime
| | | +- io.grpc:grpc-core:jar:1.65.0:compile
| | | | +- com.google.code.gson:gson:jar:2.10.1:runtime
| | | | +- com.google.android:annotations:jar:4.1.1.4:runtime
| | | | \- io.grpc:grpc-context:jar:1.65.0:runtime
| | | +- io.perfmark:perfmark-api:jar:0.26.0:runtime
| | | \- io.grpc:grpc-api:jar:1.65.0:compile
| | +- io.grpc:grpc-protobuf:jar:1.65.0:compile
| | | +- com.google.api.grpc:proto-google-common-protos:jar:2.29.0:compile
| | | \- io.grpc:grpc-protobuf-lite:jar:1.65.0:runtime
| | +- io.grpc:grpc-stub:jar:1.65.0:compile
| | +- org.apache.yetus:audience-annotations:jar:0.15.0:compile
| | +- org.mindrot:jbcrypt:jar:0.4:compile
| | +- com.github.seancfoley:ipaddress:jar:5.5.0:compile
| | +- com.yscope.clp:clp-ffi:jar:0.4.4:compile
| | +- io.github.hakky54:sslcontext-kickstart-for-netty:jar:8.3.6:compile
| | | \- io.github.hakky54:sslcontext-kickstart:jar:8.3.6:compile
| | \- com.google.re2j:re2j:jar:1.7:compile
| +- org.asynchttpclient:async-http-client:jar:2.12.3:compile
| | +- org.asynchttpclient:async-http-client-netty-utils:jar:2.12.3:compile
| | | \- io.netty:netty-buffer:jar:4.1.60.Final:compile
| | +- io.netty:netty-codec-http:jar:4.1.60.Final:compile
| | | +- io.netty:netty-common:jar:4.1.60.Final:compile
| | | +- io.netty:netty-transport:jar:4.1.60.Final:compile
| | | \- io.netty:netty-codec:jar:4.1.60.Final:compile
| | +- io.netty:netty-handler:jar:4.1.60.Final:compile
| | | \- io.netty:netty-resolver:jar:4.1.60.Final:compile
| | +- io.netty:netty-codec-socks:jar:4.1.60.Final:compile
| | +- io.netty:netty-handler-proxy:jar:4.1.60.Final:compile
| | +- io.netty:netty-transport-native-epoll:jar:linux-x86_64:4.1.60.Final:compile
| | | \- io.netty:netty-transport-native-unix-common:jar:4.1.60.Final:compile
| | +- io.netty:netty-transport-native-kqueue:jar:osx-x86_64:4.1.60.Final:compile
| | +- org.reactivestreams:reactive-streams:jar:1.0.3:compile
| | +- com.typesafe.netty:netty-reactive-streams:jar:2.0.4:compile
| | \- com.sun.activation:jakarta.activation:jar:1.2.2:compile
| +- com.101tec:zkclient:jar:0.11:compile
| \- org.apache.pinot:pinot-spi:jar:1.2.0:compile
| +- org.apache.commons:commons-configuration2:jar:2.11.0:compile
| +- commons-codec:commons-codec:jar:1.17.0:compile
| +- org.yaml:snakeyaml:jar:2.2:compile
| +- org.apache.avro:avro:jar:1.11.3:compile
| \- org.codehaus.groovy:groovy-all:jar:2.4.21:compile
```
And as a user the client works for me with the following exclusions:
```
<dependency>
<groupId>org.apache.pinot</groupId>
<artifactId>pinot-java-client</artifactId>
<exclusions>
<exclusion>
<groupId>org.glassfish.jersey.containers</groupId>
<artifactId>jersey-container-grizzly2-http</artifactId>
</exclusion>
<exclusion>
<groupId>org.glassfish.jersey.media</groupId>
<artifactId>jersey-media-multipart</artifactId>
</exclusion>
<exclusion>
<groupId>org.glassfish.jersey.media</groupId>
<artifactId>jersey-media-json-jackson</artifactId>
</exclusion>
<exclusion>
<groupId>org.glassfish.jersey.inject</groupId>
<artifactId>jersey-hk2</artifactId>
</exclusion>
<exclusion>
<groupId>org.glassfish.hk2</groupId>
<artifactId>hk2-metadata-generator</artifactId>
</exclusion>
<exclusion>
<groupId>io.swagger</groupId>
<artifactId>swagger-jersey2-jaxrs</artifactId>
</exclusion>
<exclusion>
<groupId>org.webjars</groupId>
<artifactId>swagger-ui</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.helix</groupId>
<artifactId>helix-core</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.zookeeper</groupId>
<artifactId>zookeeper</artifactId>
</exclusion>
<!-- these 2 dependencies are removed in pinot-java-client 1.3.0 https://github.com/apache/pinot/commit/6d64650e7c210456a890ee6f9a6eaf05a7ab557b#diff-417cfb66063e3f9b092a1acd9e0842f724d938af0451cb46ee864beb8a915198 -->
<exclusion>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-api</artifactId>
</exclusion>
<exclusion>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
</exclusion>
<exclusion>
<groupId>org.testng</groupId>
<artifactId>testng</artifactId>
</exclusion>
<exclusion>
<groupId>com.jayway.jsonpath</groupId>
<artifactId>json-path</artifactId>
</exclusion>
<exclusion>
<groupId>io.grpc</groupId>
<artifactId>grpc-protobuf</artifactId>
</exclusion>
<exclusion>
<groupId>io.grpc</groupId>
<artifactId>grpc-netty-shaded</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.yetus</groupId>
<artifactId>audience-annotations</artifactId>
</exclusion>
<exclusion>
<groupId>org.mindrot</groupId>
<artifactId>jbcrypt</artifactId>
</exclusion>
<exclusion>
<groupId>com.github.seancfoley</groupId>
<artifactId>ipaddress</artifactId>
</exclusion>
<exclusion>
<groupId>com.yscope.clp</groupId>
<artifactId>clp-ffi</artifactId>
</exclusion>
<exclusion>
<groupId>jakarta.servlet</groupId>
<artifactId>jakarta.servlet-api</artifactId>
</exclusion>
<exclusion>
<groupId>org.xerial.snappy</groupId>
<artifactId>snappy-java</artifactId>
</exclusion>
<exclusion>
<groupId>com.github.luben</groupId>
<artifactId>zstd-jni</artifactId>
</exclusion>
<exclusion>
<groupId>org.lz4</groupId>
<artifactId>lz4-java</artifactId>
</exclusion>
<exclusion>
<groupId>com.google.protobuf</groupId>
<artifactId>protobuf-java</artifactId>
</exclusion>
<exclusion>
<groupId>io.grpc</groupId>
<artifactId>grpc-stub</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-slf4j2-impl</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-1.2-api</artifactId>
</exclusion>
<!-- pinot-segment-spi is required but all dependencies can be removed -->
<exclusion>
<groupId>org.roaringbitmap</groupId>
<artifactId>RoaringBitmap</artifactId>
</exclusion>
<exclusion>
<groupId>org.roaringbitmap</groupId>
<artifactId>RoaringBitmap</artifactId>
</exclusion>
<exclusion>
<groupId>org.locationtech.jts</groupId>
<artifactId>jts-core</artifactId>
</exclusion>
<exclusion>
<groupId>org.xerial.larray</groupId>
<artifactId>larray-mmap</artifactId>
</exclusion>
<exclusion>
<groupId>net.openhft</groupId>
<artifactId>posix</artifactId>
</exclusion>
<exclusion>
<groupId>net.openhft</groupId>
<artifactId>chronicle-core</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>org.apache.pinot</groupId>
<artifactId>pinot-spi</artifactId>
<exclusions>
<exclusion>
<groupId>javax.mail</groupId>
<artifactId>mail</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-slf4j2-impl</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-1.2-api</artifactId>
</exclusion>
<exclusion>
<groupId>org.yaml</groupId>
<artifactId>snakeyaml</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.avro</groupId>
<artifactId>avro</artifactId>
</exclusion>
<exclusion>
<groupId>org.codehaus.groovy</groupId>
<artifactId>groovy-all</artifactId>
</exclusion>
<exclusion>
<groupId>org.slf4j</groupId>
<artifactId>jcl-over-slf4j</artifactId>
</exclusion>
</exclusions>
</dependency>
```
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] abhioncbr commented on issue #11507: pinot-clients have too many dependencies
Posted by "abhioncbr (via GitHub)" <gi...@apache.org>.
abhioncbr commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-1724552681
> Another approach is to remove the dependencies of `pinot-common`, only `pinot-spi`. This might take longer time to finish. We need to extract the `SqlParser` out from the `pinot-common`
I can start looking into this, if it's the approach we want to take.
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] pinot-clients have too many dependencies [pinot]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed issue #11507: pinot-clients have too many dependencies
URL: https://github.com/apache/pinot/issues/11507
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] abhioncbr commented on issue #11507: pinot-clients have too many dependencies
Posted by "abhioncbr (via GitHub)" <gi...@apache.org>.
abhioncbr commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-1705573227
I can look into it, to make it slim. 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.
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] pinot-clients have too many dependencies [pinot]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-5824547553
This issue has been automatically marked as stale because it has not had any activity in the last 150 days. It will be closed in 30 days if no further activity occurs. If this issue is still relevant, please leave a comment to keep it open. Thank you for your contributions!
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] xiangfu0 commented on issue #11507: pinot-clients have too many dependencies
Posted by "xiangfu0 (via GitHub)" <gi...@apache.org>.
xiangfu0 commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-1724901472
Some improvement: https://github.com/apache/pinot/pull/11620
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] pinot-clients have too many dependencies [pinot]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #11507:
URL: https://github.com/apache/pinot/issues/11507#issuecomment-6405466815
This issue has been closed because it has been inactive for 180 days. If you believe this issue is still relevant, please feel free to reopen it or create a new issue. Thank you!
--
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: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org