mumrah commented on PR #18697: URL: https://github.com/apache/kafka/pull/18697#issuecomment-2704705840
Actually, I think this won't work. Even if we clean out server-common dependencies from test-common-internal-api, the clients module (and other Java 11 modules) can't bring in test-common-runtime. Not even as a runtime-only dependency. I think the only solution here is to create a new integration test module for core similar to what streams did. @ijuma / @chia7712 WDYT about adding ":clients:integration-tests" as a new Java 17 module? This would depend on ":clients" and the test infra and would let us move a bunch of tests out of ":core" -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
