Skip to content

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

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

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

Fixed the cluster tests

6c9c5a7
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Nov 5, 2026 in 1m 8s

Quality Gate failed

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 774 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-ChLLDlydGVG&open=AaGEW9gU-ChLLDlydGVG&pullRequest=3261

Check warning on line 133 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 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=AaGEW9gd-ChLLDlydGVW&open=AaGEW9gd-ChLLDlydGVW&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 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 575 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 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=AaGEW9gg-ChLLDlydGVf&open=AaGEW9gg-ChLLDlydGVf&pullRequest=3261

Check warning on line 487 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

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

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AaGEW9ga-ChLLDlydGVR&open=AaGEW9ga-ChLLDlydGVR&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 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 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 140 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/stream/SlotCursor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AaGEW9gb-ChLLDlydGVV&open=AaGEW9gb-ChLLDlydGVV&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 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 580 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 577, which has the same body. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AaGEW9gU-ChLLDlydGU9&open=AaGEW9gU-ChLLDlydGU9&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 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 350 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 byte[]" prefix; the declaration already specifies the array type. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AaGEW9gg-ChLLDlydGVe&open=AaGEW9gg-ChLLDlydGVe&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 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 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 1126 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 1123, which has the same body. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AaGEW9gU-ChLLDlydGVM&open=AaGEW9gU-ChLLDlydGVM&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 593 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

Replace this lambda with method reference 'ValueWriter::writeNull'. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AaGEW9gg-ChLLDlydGVg&open=AaGEW9gg-ChLLDlydGVg&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 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 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