Skip to content

[client-v2] Performance #2620

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

Draft
chernser wants to merge 1 commit into
base: main
Choose a base branch
from

done a little optimization

866e6ef
Select commit
Loading
Failed to load commit list.
Draft

[client-v2] Performance #2620

done a little optimization
866e6ef
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Dec 9, 2025 in 1m 11s

Quality Gate failed

Failed conditions
69.0% Coverage on New Code (required ≥ 80%)
4.3% Duplication on New Code (required ≤ 3%)
E 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 warning on line 84 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 482 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 739 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 360 in clickhouse-data/src/test/java/com/clickhouse/data/value/ClickHouseLongValueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Swap these 2 arguments so they are in the correct order: expected value, actual value. 

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

Check warning on line 329 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 78 in jdbc-v2/src/test/java/com/clickhouse/jdbc/internal/SqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

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

Check warning on line 485 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 224 in jdbc-v2/src/main/java/com/clickhouse/jdbc/internal/JdbcUtils.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 75 to 64, Complexity from 56 to 14, Nesting Level from 3 to 2, Number of Variables from 10 to 6. 

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

Check warning on line 87 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 703 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'com.clickhouse.jdbc.parser.ParseException', as it cannot be thrown from method's body. 

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

Check warning on line 439 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 128 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 235 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 443 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 505 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 1278 in jdbc-v2/src/main/java/com/clickhouse/jdbc/metadata/DatabaseMetaDataImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Complete the task associated to this TODO comment. 

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

Check warning on line 353 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 121 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 620 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'com.clickhouse.jdbc.parser.ParseException', as it cannot be thrown from method's body. 

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

Check warning on line 480 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 471 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

Check warning on line 1272 in jdbc-v2/src/main/java/com/clickhouse/jdbc/metadata/DatabaseMetaDataImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Complete the task associated to this TODO comment. 

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

Check warning on line 178 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof TemporalAccessor temporalaccessor' 

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

Check warning on line 325 in clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/parser/ClickHouseSqlParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "StatementType"; it is deprecated. 

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