Skip to content

[repo] Fix slow init/teardown in cloud tests #2852

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

Merged
chernser merged 10 commits into from
May 15, 2026
+65 −16

Added some protective code here and there

6486bd9
Select commit
Loading
Failed to load commit list.
Merged

[repo] Fix slow init/teardown in cloud tests #2852

Added some protective code here and there
6486bd9
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 15, 2026 in 58s

Quality Gate passed

Issues
6 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 15 in jdbc-v2/src/test/java/com/clickhouse/jdbc/JdbcIntegrationTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.util.List'. 

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

Check warning on line 337 in clickhouse-client/src/test/java/com/clickhouse/client/ClickHouseServerForTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge this if statement with the enclosing one. 

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

Check failure on line 269 in jdbc-v2/src/main/java/com/clickhouse/jdbc/internal/JdbcConfiguration.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check failure on line 65 in clickhouse-client/src/test/java/com/clickhouse/client/ClickHouseServerForTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. 

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

Check warning on line 7 in jdbc-v2/src/test/java/com/clickhouse/jdbc/JdbcIntegrationTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'com.clickhouse.client.api.query.GenericRecord'. 

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

Check warning on line 375 in clickhouse-client/src/test/java/com/clickhouse/client/ClickHouseServerForTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge this if statement with the enclosing one. 

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