Skip to content

Fix client-v2: serialize SimpleAggregateFunction columns in RowBinary writer#2937

Open
polyglotAI-bot wants to merge 2 commits into
mainfrom
polyglot/rowbinary-simpleaggregatefunction-writer
Open

Fix client-v2: serialize SimpleAggregateFunction columns in RowBinary writer#2937
polyglotAI-bot wants to merge 2 commits into
mainfrom
polyglot/rowbinary-simpleaggregatefunction-writer

Address review: clarify serializeNestedData SAF exception, fix CHANGE…

159f43a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 19, 2026 in 1m 22s

Quality Gate failed

Failed conditions
75.0% Coverage on New Code (required ≥ 80%)
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 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=AZ96cbijXCM0T390Hzzq&open=AZ96cbijXCM0T390Hzzq&pullRequest=2937

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=AZ9ybs5YDeeK920-90e5&open=AZ9ybs5YDeeK920-90e5&pullRequest=2937
View more details on SonarQubeCloud