gharris1727 commented on code in PR #13178:
URL: https://github.com/apache/kafka/pull/13178#discussion_r1100680887
##########
connect/mirror/src/test/java/org/apache/kafka/connect/mirror/integration/MirrorConnectorsIntegrationBaseTest.java:
##########
@@ -503,25 +495,6 @@ public void testOffsetSyncsTopicsOnTarget() throws
Exception {
produceMessages(primary, "test-topic-1");
- ReplicationPolicy replicationPolicy = new
MirrorClient(mm2Config.clientConfig(BACKUP_CLUSTER_ALIAS)).replicationPolicy();
- String remoteTopic =
replicationPolicy.formatRemoteTopic(PRIMARY_CLUSTER_ALIAS, "test-topic-1");
-
- // Check offsets are pushed to the checkpoint topic
- Consumer<byte[], byte[]> backupConsumer =
backup.kafka().createConsumerAndSubscribeTo(Collections.singletonMap(
- "auto.offset.reset", "earliest"), PRIMARY_CLUSTER_ALIAS +
".checkpoints.internal");
- waitForCondition(() -> {
- ConsumerRecords<byte[], byte[]> records =
backupConsumer.poll(Duration.ofSeconds(1L));
- for (ConsumerRecord<byte[], byte[]> record : records) {
- Checkpoint checkpoint = Checkpoint.deserializeRecord(record);
- if (remoteTopic.equals(checkpoint.topicPartition().topic())) {
- return true;
- }
- }
- return false;
- }, 30_000,
- "Unable to find checkpoints for " + PRIMARY_CLUSTER_ALIAS +
".test-topic-1"
- );
Review Comment:
I think that makes sense.
I'll add some explicitly committed offsets here as a quick fix, and re-add
the assertion.
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
