gharris1727 commented on code in PR #13165: URL: https://github.com/apache/kafka/pull/13165#discussion_r1203138122
########## connect/runtime/src/main/java/org/apache/kafka/connect/cli/AbstractConnectCli.java: ########## @@ -119,36 +120,37 @@ public Connect startConnect(Map<String, String> workerProps, String... extraArgs log.info("Scanning for plugin classes. This might take a moment ..."); Plugins plugins = new Plugins(workerProps); - plugins.compareAndSwapWithDelegatingLoader(); - T config = createConfig(workerProps); - log.debug("Kafka cluster ID: {}", config.kafkaClusterId()); + try (LoaderSwap loaderSwap = plugins.withClassLoader(plugins.delegatingLoader())) { Review Comment: Also here is some of the context for this change: https://github.com/apache/kafka/pull/13165#discussion_r1161929533 Since the elimination of compareAndSwap is technically unrelated to the title change, it could be moved out to it's own PR. Let me know if you'd like me to separate the two changes. -- 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