gharris1727 opened a new pull request, #13313:
URL: https://github.com/apache/kafka/pull/13313

   Currently connect-runtime pulls in tools as a dependency in order to use the 
ThroughputThrottler.
   In the future it will be desirable to have tools pull in connect-runtime as 
a (test) dependency, so we must remove this dependency first.
   
   The last time this was tried 
(https://issues.apache.org/jira/browse/KAFKA-2752) it broke system tests and 
had to be reverted (https://issues.apache.org/jira/browse/KAFKA-2807).
   
   This was because the `dev` version of tools was being used with `0.8.2.x` 
clients jar in the system tests, because the test needs the VerifiableProducer 
which did not exist in `0.8.2.x`. In order to work around this restriction, 
this PR changes the system test to use the `0.9.x` version of tools with 
`0.8.2.x` instead of the dev version. This will relax the current compatibility 
requirement between tools and clients, and should allow this refactor to land 
without the upgrade test failing.
   
   This conflicts with #13302 which move some of the classes which depend on 
`tools`.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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

Reply via email to