Skip to content

Fix client-v2: quote String/temporal elements in container query parameters#2898

Merged
chernser merged 5 commits into
mainfrom
polyglot/fix-container-query-param-quoting
Jul 8, 2026
Merged

Fix client-v2: quote String/temporal elements in container query parameters#2898
chernser merged 5 commits into
mainfrom
polyglot/fix-container-query-param-quoting

Use an explicit stack for container query-param formatting

c17591f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 8, 2026 in 1m 4s

Quality Gate failed

Failed conditions
C 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 205 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOUE&open=AZ8FXGKzw7IBCOp2OOUE&pullRequest=2898

Check warning on line 186 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOUB&open=AZ8FXGKzw7IBCOp2OOUB&pullRequest=2898

Check warning on line 185 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOUA&open=AZ8FXGKzw7IBCOp2OOUA&pullRequest=2898

Check warning on line 1670 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8E20qMI9QKIFBcMBx2&open=AZ8E20qMI9QKIFBcMBx2&pullRequest=2898

Check failure on line 75 in .github/workflows/release.yml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use full commit SHA hash for this dependency. 

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

Check failure on line 64 in .github/workflows/release.yml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use full commit SHA hash for this dependency. 

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

Check warning on line 183 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOT_&open=AZ8FXGKzw7IBCOp2OOT_&pullRequest=2898

Check warning on line 2253 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "spy". 

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

Check warning on line 1672 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ9DypNZR51oomk0lMeX&open=AZ9DypNZR51oomk0lMeX&pullRequest=2898

Check warning on line 1669 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ9DypNZR51oomk0lMeW&open=AZ9DypNZR51oomk0lMeW&pullRequest=2898

Check warning on line 201 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOUD&open=AZ8FXGKzw7IBCOp2OOUD&pullRequest=2898

Check warning on line 183 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOT-&open=AZ8FXGKzw7IBCOp2OOT-&pullRequest=2898

Check warning on line 2255 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "times". 

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

Check failure on line 54 in .github/workflows/release.yml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use full commit SHA hash for this dependency. 

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

Check failure on line 287 in client-v2/src/main/java/com/clickhouse/client/api/internal/DataTypeConverter.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=AZ9D_Hqy_UG81LbkguPx&open=AZ9D_Hqy_UG81LbkguPx&pullRequest=2898

Check warning on line 18 in .github/workflows/release.yml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace "write-all" with specific permissions (e.g., "contents: write"). 

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

Check warning on line 197 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOUC&open=AZ8FXGKzw7IBCOp2OOUC&pullRequest=2898

Check warning on line 175 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ8FXGKzw7IBCOp2OOT9&open=AZ8FXGKzw7IBCOp2OOT9&pullRequest=2898

Check warning on line 228 in client-v2/src/test/java/com/clickhouse/client/api/internal/DataTypeConverterTest.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ9D_Ho0_UG81LbkguPw&open=AZ9D_Ho0_UG81LbkguPw&pullRequest=2898

Check warning on line 2255 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify". 

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

Check warning on line 1677 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ9DypNZR51oomk0lMeY&open=AZ9DypNZR51oomk0lMeY&pullRequest=2898

Check warning on line 1669 in client-v2/src/test/java/com/clickhouse/client/query/QueryTests.java

See this annotation in the file changed.

@sonarqubecloud 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=AZ9DypNZR51oomk0lMeV&open=AZ9DypNZR51oomk0lMeV&pullRequest=2898
View more details on SonarQubeCloud