Skip to content

feat(client-v2): decode QBit in the Native format - #2983

Open
polyglotAI-bot wants to merge 4 commits into
mainfrom
polyglot/client-v2-qbit-native-read
Open

feat(client-v2): decode QBit in the Native format#2983
polyglotAI-bot wants to merge 4 commits into
mainfrom
polyglot/client-v2-qbit-native-read

Address review: raise QBit Native decode coverage, shorten comments, …

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

Annotations

Check failure on line 676 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/BinaryStreamReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check failure on line 1502 in client-v2/src/main/java/com/clickhouse/client/api/Client.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Retrying." 3 times.

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

Check warning on line 761 in jdbc-v2/src/main/java/com/clickhouse/jdbc/PreparedStatementImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a primitive boolean expression here.

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

Check warning on line 319 in client-v2/src/test/java/com/clickhouse/client/datatypes/DataTypeTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "settings" which hides the field declared at line 60.

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

Check warning on line 270 in client-v2/src/test/java/com/clickhouse/client/datatypes/DataTypeTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "settings" which hides the field declared at line 60.

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

Check warning on line 180 in client-v2/src/main/java/com/clickhouse/client/api/internal/ClickHouseLZ4InputStream.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this method to simply inherit it.

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-0WFgNHYkM0naiCcRX&open=AZ-0WFgNHYkM0naiCcRX&pullRequest=2983