Joker-5 commented on code in PR #15642:
URL: https://github.com/apache/kafka/pull/15642#discussion_r1552049826
##########
connect/runtime/src/test/java/org/apache/kafka/connect/integration/ConnectorValidationIntegrationTest.java:
##########
@@ -69,6 +69,9 @@ public static void setup() {
Map<String, String> workerProps = new HashMap<>();
workerProps.put(GROUP_ID_CONFIG, WORKER_GROUP_ID);
+ // Work around a circular-dependency in TestPlugins.
+ TestPlugins.pluginPath();
Review Comment:
> Thanks for the explanation. hmm I ran the tests in
`ConnectorValidationIntergationTest` without this one line but including all
others and the tests are passing for me locally. I created a branch (draft, we
can close it once we have validated the behaviour) #15653 with the changes. We
can monitor if the circular dependency related errors show up there or not.
@vamossagar12, I checked the result from #15653 which did not have related
errors, so I removed this one line.
Maybe there's some problems in my local env, I will try to investigate the
root cause :)
--
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
