You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by "ruanwenjun (via GitHub)" <gi...@apache.org> on 2023/09/14 06:22:02 UTC

[GitHub] [seatunnel] ruanwenjun commented on a diff in pull request #5476: [Improvement] [clickhouse-e2e] Enhance ClickHouse E2E testing to trigger multiple checkpoints

ruanwenjun commented on code in PR #5476:
URL: https://github.com/apache/seatunnel/pull/5476#discussion_r1325413594


##########
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-clickhouse-e2e/src/test/java/org/apache/seatunnel/connectors/seatunnel/clickhouse/ClickhouseIT.java:
##########
@@ -76,7 +76,7 @@
 
 public class ClickhouseIT extends TestSuiteBase implements TestResource {
     private static final Logger LOG = LoggerFactory.getLogger(ClickhouseIT.class);
-    private static final String CLICKHOUSE_DOCKER_IMAGE = "yandex/clickhouse-server:22.1.3.7";
+    private static final String CLICKHOUSE_DOCKER_IMAGE = "clickhouse/clickhouse-server:latest";

Review Comment:
   It's not a good idea to change the image to the latest version. Please revert this.



-- 
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: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org