clickhouse-java icon indicating copy to clipboard operation
clickhouse-java copied to clipboard

[client-v2] Make compression dependencies as required

Open chernser opened this issue 1 year ago • 1 comments

Summary

Client uses compression by default, but related library dependencies are optional. It causes confusion because users have to add extra dependencies while library cannot work without it without adjustments. This PR fixes:

  • compression dependencies are required.
  • apache http client dependencies are cleaned up
  • updated in examples

Currently dependencies look like:

productionRuntimeClasspath
+--- com.clickhouse:client-v2:0.7.0-SNAPSHOT
|    +--- com.clickhouse:clickhouse-client:0.7.0-SNAPSHOT
|    +--- com.clickhouse:clickhouse-data:0.7.0-SNAPSHOT
|    +--- org.slf4j:slf4j-api:2.0.7 -> 2.0.13
|    +--- org.apache.httpcomponents.client5:httpclient5:5.3.1
|    +--- org.lz4:lz4-pure-java:1.8.0
|    +--- org.apache.commons:commons-compress:1.27.1
|    +--- commons-codec:commons-codec:1.17.1 -> 1.16.1
|    +--- commons-io:commons-io:2.16.1
|    +--- org.apache.commons:commons-lang3:3.16.0 -> 3.14.0
|    +--- org.ow2.asm:asm:9.7
|    \--- com.fasterxml.jackson.core:jackson-core:2.17.2

Closes: https://github.com/ClickHouse/clickhouse-java/issues/1805 Closes: https://github.com/ClickHouse/clickhouse-java/issues/1853

Checklist

Delete items not relevant to your PR:

  • [ ] Unit and integration tests covering the common scenarios were added
  • [ ] A human-readable description of the changes was provided to include in CHANGELOG
  • [ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

chernser avatar Oct 09 '24 06:10 chernser