[jdbc-v2] Added test for WITH statement with parameters.

[chernser]

Summary

• Adds test for WITH clause with parameters to PreparedStatementImplTests

Closes #2391

Checklist

Delete items not relevant to your PR:

• Closes #
• Unit and integration tests covering the common scenarios were added
• A human-readable description of the changes was provided to include in CHANGELOG
• For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

[sonarqubecloud]

Quality Gate failed

Failed conditions
66.7% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[Copilot]

Pull Request Overview

This PR adds integration tests for the WITH clause using parameters in prepared statements and cleans up unused overrides in several JDBC classes.

• Added a new test, testWithClauseWithParams, in PreparedStatementTest.java to validate parameterized WITH statements.
• Removed redundant overridden methods from StatementImpl, DataSourceImpl, and ConnectionImpl to streamline their implementations.

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

File	Description
jdbc-v2/src/test/java/com/clickhouse/jdbc/PreparedStatementTest.java	Added tests for WITH clause and improved test structure by scoping variable declarations.
jdbc-v2/src/main/java/com/clickhouse/jdbc/StatementImpl.java	Removed unused override methods to rely on default implementations.
jdbc-v2/src/main/java/com/clickhouse/jdbc/DataSourceImpl.java	Removed unused methods and corresponding imports related to connection building and sharding.
jdbc-v2/src/main/java/com/clickhouse/jdbc/ConnectionImpl.java	Revised the array creation implementation and removed deprecated sharding key methods.

Files not reviewed (2)

• clickhouse-client/pom.xml: Language not supported
• pom.xml: Language not supported

Comments suppressed due to low confidence (2)

jdbc-v2/src/test/java/com/clickhouse/jdbc/PreparedStatementTest.java:332

• Consider using a consistent method (either column labels or column indexes) for accessing result set values to improve clarity and reduce the risk of issues if the column order changes.

Assert.assertEquals(rs.getTimestamp("target_time").toLocalDateTime().truncatedTo(ChronoUnit.SECONDS), target_time.toInstant().atZone(ZoneId.of("UTC")).toLocalDateTime().truncatedTo(ChronoUnit.SECONDS));

jdbc-v2/src/main/java/com/clickhouse/jdbc/ConnectionImpl.java:553

• [nitpick] If there's no specific need for streaming the elements, consider using Arrays.asList(elements) to simplify the code and improve readability, assuming that a direct conversion is acceptable.

List<Object> list = (elements == null || elements.length == 0) ? Collections.emptyList() : Arrays.stream(elements, 0, elements.length).collect(Collectors.toList());

[mshustov]

/windsurf-review