[Improvement] [clickhouse-e2e] Enhance ClickHouse E2E testing to trigger multiple checkpoints #5476
Conversation
| 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"; |
There was a problem hiding this comment.
It's not a good idea to change the image to the latest version.
There was a problem hiding this comment.
@ruanwenjun ClickhouseSinkCDCChangelogIT uses clickhouse/clickhouse-server:latest. Should we unify it to yandex/clickhouse-server:22.1.3.7
There was a problem hiding this comment.
I think @ruanwenjun means don't use latest, the version it represents can be changed at any time. Just use a certain version.
ClickhouseSinkCDCChangelogIT uses clickhouse/clickhouse-server:latest. Should we unify it to yandex/clickhouse-server:22.1.3.7
Just unify use version which clickhouse/clickhouse-server:latest represents. (Seem like is 23.3.13.6)
There was a problem hiding this comment.
Just unify use version which
clickhouse/clickhouse-server:latestrepresents. (Seem like is23.3.13.6)
ok，I'll replace it with that version
ruanwenjun
changed the title
…ger multiple checkpoints (apache#5476)
3 participants
Enhance ClickHouse E2E testing to trigger multiple checkpoints
#5468
Purpose of this pull request
Check list
New License Guide
release-note.