gharris1727 commented on code in PR #16525:
URL: https://github.com/apache/kafka/pull/16525#discussion_r1669065888
##########
connect/mirror/src/test/java/org/apache/kafka/connect/mirror/integration/DedicatedMirrorIntegrationTest.java:
##########
@@ -354,8 +355,10 @@ private <T extends SourceConnector> void
awaitTaskConfigurations(MirrorMaker mm,
.map(TaskInfo::config)
.allMatch(predicate);
} catch (Exception ex) {
Review Comment:
I think this was over-general in the first place, causing the typo. It
should be an ExecutionException, and the retriable condition should be
simplified. I don't see anything in this method that would throw a synchronous
RetriableException.
--
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
