Skip to content

[client-v2] Value Reader/Writer API - #3261

Draft
chernser wants to merge 2 commits into
mainfrom
09/14/26/reader_writer_api
Draft

[client-v2] Value Reader/Writer API #3261
chernser wants to merge 2 commits into
mainfrom
09/14/26/reader_writer_api

Merge branch 'main' into 09/14/26/reader_writer_api

e0bf26b
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Oct 28, 2026 in 1m 2s

Quality Gate failed

Failed conditions
45.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 61 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

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

Check warning on line 738 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "index" parameter 

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

Check warning on line 951 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 948, which has the same body. 

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

Check warning on line 768 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "index" parameter 

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

Check warning on line 150 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamRoundTripTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

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

Check warning on line 744 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "index" parameter 

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

Check warning on line 667 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 664, which has the same body. 

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

Check warning on line 756 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "index" parameter 

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

Check warning on line 780 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "index" parameter 

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

Check warning on line 820 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 816, which has the same body. 

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

Check warning on line 313 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamReaderScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this if-then-else statement by a single return statement. 

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

Check warning on line 1166 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 1163, which has the same body. 

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

Check warning on line 269 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamServerTest.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=AaGEW9gd-ChLLDlydGVY&open=AaGEW9gd-ChLLDlydGVY&pullRequest=3261

Check warning on line 225 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamRoundTripTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

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

Check warning on line 980 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 977, which has the same body. 

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

Check warning on line 688 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

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

Check warning on line 95 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamTypes.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Return an empty collection instead of null. 

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

Check warning on line 12 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamReaderScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.io.EOFException'. 

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

Check warning on line 194 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 192, which has the same body. 

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

Check warning on line 915 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this catch with the one at line 912, which has the same body. 

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

Check warning on line 20 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamRoundTripTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.nio.charset.StandardCharsets'. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 750 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/StreamWriterScope.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "index" parameter 

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

Check warning on line 162 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamRoundTripTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

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

Check warning on line 298 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/stream/RowBinaryStreamRoundTripTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the "new int[]" prefix; the declaration already specifies the array type. 

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