Skip to content

[client-v2] Request cancellation#2903

Open
chernser wants to merge 12 commits into
mainfrom
06/26/26/cancel_http_request
Open

[client-v2] Request cancellation#2903
chernser wants to merge 12 commits into
mainfrom
06/26/26/cancel_http_request

Fixed unregistering request. Added tests that request is not retried

edb40f4
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 1, 2026 in 1m 7s

Quality Gate passed

Issues
10 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

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=AZ8V6sl3aauKZWBXEhrr&open=AZ8V6sl3aauKZWBXEhrr&pullRequest=2903

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=AZ8V6sl3aauKZWBXEhrq&open=AZ8V6sl3aauKZWBXEhrq&pullRequest=2903

Check warning on line 589 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=AZ8V6siVaauKZWBXEhro&open=AZ8V6siVaauKZWBXEhro&pullRequest=2903

Check warning on line 560 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=AZ8V6siVaauKZWBXEhrn&open=AZ8V6siVaauKZWBXEhrn&pullRequest=2903

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=AZ8V6sl3aauKZWBXEhrp&open=AZ8V6sl3aauKZWBXEhrp&pullRequest=2903

Check failure on line 671 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=AZ8bl0N8HYxAQGlrmq2g&open=AZ8bl0N8HYxAQGlrmq2g&pullRequest=2903

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=AZ8V6siVaauKZWBXEhrm&open=AZ8V6siVaauKZWBXEhrm&pullRequest=2903

Check warning on line 661 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=AZ8bl0N8HYxAQGlrmq2f&open=AZ8bl0N8HYxAQGlrmq2f&pullRequest=2903

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=AZ8V6siVaauKZWBXEhrk&open=AZ8V6siVaauKZWBXEhrk&pullRequest=2903

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=AZ8V6siVaauKZWBXEhrl&open=AZ8V6siVaauKZWBXEhrl&pullRequest=2903
View more details on SonarQubeCloud