Skip to content

✨[client-v2, jdbc-v2] Allow setting http path in URL #2691

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 4 commits into
base: main
Choose a base branch
from

some code cleanup

bffb301
Select commit
Loading
Failed to load commit list.
Open

✨[client-v2, jdbc-v2] Allow setting http path in URL #2691

some code cleanup
bffb301
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Dec 12, 2025 in 1m 0s

Quality Gate passed

Issues
5 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 994 in jdbc-v2/src/test/java/com/clickhouse/jdbc/ConnectionTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move the array designators [] to the type. 

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

Check warning on line 983 in jdbc-v2/src/test/java/com/clickhouse/jdbc/ConnectionTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move the array designators [] to the type. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "connectionUrl" which hides the field declared at line 48. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 71 to 64, Complexity from 24 to 14, Nesting Level from 4 to 2, Number of Variables from 17 to 6. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this hard-coded path-delimiter. 

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