Skip to content

Sync/release 0.10.0: Request Cancellation#2943

Open
chernser wants to merge 19 commits into
mainfrom
sync/release_0.10.0
Open

Sync/release 0.10.0: Request Cancellation#2943
chernser wants to merge 19 commits into
mainfrom
sync/release_0.10.0

fixed skipped cancellation check

26bbf39
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 21, 2026 in 1m 45s

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check failure on line 699 in client-v2/src/main/java/com/clickhouse/client/api/internal/HttpAPIClientHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this "try" to a try-with-resources. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DblpJhtaNZptrj2xv&open=AZ-DblpJhtaNZptrj2xv&pullRequest=2943

Check warning on line 2118 in client-v2/src/test/java/com/clickhouse/client/HttpTransportTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbloMhtaNZptrj2xr&open=AZ-DbloMhtaNZptrj2xr&pullRequest=2943

Check warning on line 293 in client-v2/src/test/java/com/clickhouse/client/api/transport/TransportBaseTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "client" parameter 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbljuhtaNZptrj2xd&open=AZ-DbljuhtaNZptrj2xd&pullRequest=2943

Check warning on line 2775 in client-v2/src/test/java/com/clickhouse/client/HttpTransportTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbloMhtaNZptrj2xs&open=AZ-DbloMhtaNZptrj2xs&pullRequest=2943

Check warning on line 301 in client-v2/src/test/java/com/clickhouse/client/api/transport/TransportBaseTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "client" parameter 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbljuhtaNZptrj2xe&open=AZ-DbljuhtaNZptrj2xe&pullRequest=2943

Check warning on line 17 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xi&open=AZ-DbllqhtaNZptrj2xi&pullRequest=2943

Check warning on line 2806 in client-v2/src/test/java/com/clickhouse/client/HttpTransportTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbloMhtaNZptrj2xt&open=AZ-DbloMhtaNZptrj2xt&pullRequest=2943

Check warning on line 689 in client-v2/src/main/java/com/clickhouse/client/api/internal/HttpAPIClientHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace generic exceptions with specific library exceptions or a custom exception. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DblpJhtaNZptrj2xu&open=AZ-DblpJhtaNZptrj2xu&pullRequest=2943

Check warning on line 588 in client-v2/src/test/java/com/clickhouse/client/api/transport/TransportBaseTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbljuhtaNZptrj2xg&open=AZ-DbljuhtaNZptrj2xg&pullRequest=2943

Check warning on line 40 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xp&open=AZ-DbllqhtaNZptrj2xp&pullRequest=2943

Check warning on line 18 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doThrow". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xj&open=AZ-DbllqhtaNZptrj2xj&pullRequest=2943

Check warning on line 24 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xl&open=AZ-DbllqhtaNZptrj2xl&pullRequest=2943

Check warning on line 559 in client-v2/src/test/java/com/clickhouse/client/api/transport/TransportBaseTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbljuhtaNZptrj2xf&open=AZ-DbljuhtaNZptrj2xf&pullRequest=2943

Check warning on line 30 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xn&open=AZ-DbllqhtaNZptrj2xn&pullRequest=2943

Check warning on line 23 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xk&open=AZ-DbllqhtaNZptrj2xk&pullRequest=2943

Check warning on line 34 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xo&open=AZ-DbllqhtaNZptrj2xo&pullRequest=2943

Check warning on line 29 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xm&open=AZ-DbllqhtaNZptrj2xm&pullRequest=2943

Check warning on line 121 in client-v2/src/test/java/com/clickhouse/client/InsertExceptionClassificationTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DblmFhtaNZptrj2xq&open=AZ-DblmFhtaNZptrj2xq&pullRequest=2943

Check warning on line 287 in client-v2/src/test/java/com/clickhouse/client/api/transport/TransportBaseTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "client" parameter 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbljuhtaNZptrj2xc&open=AZ-DbljuhtaNZptrj2xc&pullRequest=2943

Check warning on line 15 in client-v2/src/test/java/com/clickhouse/client/api/internal/ClientUtilsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DbllqhtaNZptrj2xh&open=AZ-DbllqhtaNZptrj2xh&pullRequest=2943

Check failure on line 404 in client-v2/src/main/java/com/clickhouse/client/api/internal/HttpAPIClientHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Fix this access that will throw a NullPointerException when executed. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-DblpJhtaNZptrj2xw&open=AZ-DblpJhtaNZptrj2xw&pullRequest=2943
View more details on SonarQubeCloud