Copilot commented on code in PR #35476:
URL: https://github.com/apache/shardingsphere/pull/35476#discussion_r2104070827
##########
test/native/src/test/resources/test-native/yaml/jdbc/databases/clickhouse.yaml:
##########
@@ -18,22 +18,16 @@
dataSources:
ds_0:
dataSourceClassName: com.zaxxer.hikari.HikariDataSource
- driverClassName: com.clickhouse.jdbc.ClickHouseDriver
- jdbcUrl: $${fixture.test-native.yaml.database.clickhouse.ds0.jdbc-url::}
- username: default
- password:
+ driverClassName: org.testcontainers.jdbc.ContainerDatabaseDriver
+ jdbcUrl:
jdbc:tc:clickhouse:25.4.5.24:///demo_ds_0?TC_INITSCRIPT=test-native/sql/clickhouse-init.sql
ds_1:
dataSourceClassName: com.zaxxer.hikari.HikariDataSource
- driverClassName: com.clickhouse.jdbc.ClickHouseDriver
- jdbcUrl: $${fixture.test-native.yaml.database.clickhouse.ds1.jdbc-url::}
- username: default
- password:
+ driverClassName: org.testcontainers.jdbc.ContainerDatabaseDriver
+ jdbcUrl:
jdbc:tc:clickhouse:25.4.5.24:///demo_ds_1?TC_INITSCRIPT=test-native/sql/clickhouse-init.sql
ds_2:
dataSourceClassName: com.zaxxer.hikari.HikariDataSource
- driverClassName: com.clickhouse.jdbc.ClickHouseDriver
- jdbcUrl: $${fixture.test-native.yaml.database.clickhouse.ds2.jdbc-url::}
- username: default
- password:
+ driverClassName: org.testcontainers.jdbc.ContainerDatabaseDriver
+ jdbcUrl:
jdbc:tc:clickhouse:25.4.5.24:///demo_ds_2?TC_INITSCRIPT=test-native/sql/clickhouse-init.sql
Review Comment:
Ensure that the specified Testcontainers JDBC driver and the TC_INITSCRIPT
parameter are intended for this integration and that their configuration aligns
with the updated dependency versions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
