Skip to content

Fix client-v2: reject null into non-nullable Array in RowBinary writer - #2940

Open
polyglotAI-bot wants to merge 2 commits into
mainfrom
polyglot/rowbinary-null-non-nullable-array
Open

Fix client-v2: reject null into non-nullable Array in RowBinary writer#2940
polyglotAI-bot wants to merge 2 commits into
mainfrom
polyglot/rowbinary-null-non-nullable-array

Add DataTypeTests integration coverage for null into non-nullable Array

816f99c
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 24, 2026 in 1m 24s

Quality Gate passed

Issues
7 New issues
0 Accepted issues

Measures
0 Security Hotspots
86.9% Coverage on New Code
0.3% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

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-WFFNDEjFhSS0SYN7y&open=AZ-WFFNDEjFhSS0SYN7y&pullRequest=2940

Check warning on line 609 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-WFFO1EjFhSS0SYN70&open=AZ-WFFO1EjFhSS0SYN70&pullRequest=2940

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 154 to 64, Complexity from 72 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-WFFNDEjFhSS0SYN7z&open=AZ-WFFNDEjFhSS0SYN7z&pullRequest=2940

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-WFFQfEjFhSS0SYN71&open=AZ-WFFQfEjFhSS0SYN71&pullRequest=2940

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-WFFGWEjFhSS0SYN7v&open=AZ-WFFGWEjFhSS0SYN7v&pullRequest=2940

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-WFFLjEjFhSS0SYN7x&open=AZ-WFFLjEjFhSS0SYN7x&pullRequest=2940

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-WFFLjEjFhSS0SYN7w&open=AZ-WFFLjEjFhSS0SYN7w&pullRequest=2940
View more details on SonarQubeCloud