Skip to content

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

Open
polyglotAI-bot wants to merge 3 commits into
mainfrom
polyglot/rowbinary-nested-writer
Open

Fix client-v2: serialize Nested columns in RowBinary writer#2936
polyglotAI-bot wants to merge 3 commits into
mainfrom
polyglot/rowbinary-nested-writer

Support Nested columns in jdbc-v2 java.sql.Array getResultSet

5112ee3
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 24, 2026 in 1m 17s

Quality Gate failed

Failed conditions
D Security 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 109 in client-v2/src/test/java/com/clickhouse/client/InsertExceptionClassificationTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock". 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.apache.hc.core5.http.HttpStatus'. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 16 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/MapBackedRecord.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'com.google.common.collect.ImmutableList'. 

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

Check failure on line 306 in client-v2/src/main/java/com/clickhouse/client/api/internal/HttpAPIClientHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Enable server hostname verification on this SSL/TLS connection. 

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

Check failure on line 107 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 18 to the 15 allowed. 

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

Check warning on line 6 in client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/MapBackedRecord.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'com.clickhouse.client.api.metadata.NoSuchColumnException'. 

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

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

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 152 to 64, Complexity from 71 to 14, Nesting Level from 3 to 2, Number of Variables from 14 to 6. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace generic exceptions with specific library exceptions or a custom exception. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ-V-65kBaRu4znBJ6Re&open=AZ-V-65kBaRu4znBJ6Re&pullRequest=2936
View more details on SonarQubeCloud