gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1650067494
@fvaleri Thanks for pointing that out! I mistakenly assumed that I wasn't
going to be affecting intermediate versions, or that the location I changed was
the only place where we mixed
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1644520071
Thanks for your help Ismael!
--
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 specif
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1644153313
@ijuma Could you take another look at this? This is blocking KIP-898 that
I'm trying to get landed in time for 3.6.0. Thanks!
--
This is an automated message from the Apache Git Ser
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1644151941
I ran a full system test run:
```
SESSION REPORT (ALL TESTS)
ducktape version: 0.11.3
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1633138815
Hey @ijuma Do you have the system test results for this branch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1581417136
@ijuma Here are the results of my local test runs of test suites which use
LATEST_0_8_2 and VerifiableProducer:
* test_verifiable_producer.py: 25/25 PASS
* compatibility_test
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1577137346
@ijuma Can you review this project layering change? You reviewed the last
time it was attempted.
Thanks!
--
This is an automated message from the Apache Git Service.
To respo
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1452503148
This refactor first landed in #432 and then the current dependency graph was
set by #512
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
gharris1727 commented on PR #13313:
URL: https://github.com/apache/kafka/pull/13313#issuecomment-1448576594
I removed the checkstyle statements which allow `connect-runtime` to import
`tools`. Since this is the only dependency on `tools`, we also have an
opportunity to disallow _any_ import
