ClickHouse / clickhouse-java

ClickHouse Java Clients & JDBC Driver
https://clickhouse.com
Apache License 2.0
1.45k stars 535 forks source link

[client-v2] Make compression dependencies as required #1859

Closed chernser closed 3 weeks ago

chernser commented 1 month ago

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:

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:

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
23 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud