Fix client-v2: serialize Nested columns in RowBinary writer - #2936

Merged

chernser merged 8 commits into

mainfrom

polyglot/rowbinary-nested-writer

Jul 29, 2026

SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 29, 2026 in 1m 11s

Quality Gate passed

Issues
4 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 390 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/internal/SerializerUtilsTest.java

sonarqubecloud / SonarCloud Code Analysis

Replace these 3 tests with a single Parameterized one.

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

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

sonarqubecloud / SonarCloud Code Analysis

Use a "java.time.Month" enum constant instead of this int literal.

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

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

sonarqubecloud / SonarCloud Code Analysis

Use a "java.time.Month" enum constant instead of this int literal.

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

Check warning on line 464 in client-v2/src/test/java/com/clickhouse/client/api/data_formats/internal/SerializerUtilsTest.java

sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "testNestedNullableRoundTrip" on line 390.

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

View more details on SonarQubeCloud