Skip to content

[client-v2] Error handling #2804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
chernser wants to merge 7 commits into
base: main
Choose a base branch
from

fixed reading error message from compressed

e3fd015
Select commit
Loading
Failed to load commit list.
Open

[client-v2] Error handling #2804

fixed reading error message from compressed
e3fd015
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 2, 2026 in 1m 1s

Quality Gate failed

Failed conditions
78.9% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 1503 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=AZ0m9TEzIqNXeWr2gTBl&open=AZ0m9TEzIqNXeWr2gTBl&pullRequest=2804

Check warning on line 1605 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=AZ0m9TEzIqNXeWr2gTBm&open=AZ0m9TEzIqNXeWr2gTBm&pullRequest=2804

Check warning on line 374 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

Reduce the total number of break and continue statements in this loop to use at most one. 

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

Check warning on line 1458 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=AZ0m9TEzIqNXeWr2gTBk&open=AZ0m9TEzIqNXeWr2gTBk&pullRequest=2804

Check failure on line 367 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

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. 

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

Check failure on line 408 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

Refactor this method to reduce its Cognitive Complexity from 32 to the 15 allowed. 

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

Check warning on line 1419 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=AZ0m9TEzIqNXeWr2gTBj&open=AZ0m9TEzIqNXeWr2gTBj&pullRequest=2804

Check warning on line 408 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=AZ0m9TGxIqNXeWr2gTBo&open=AZ0m9TGxIqNXeWr2gTBo&pullRequest=2804
View more details on SonarQubeCloud