C0urante commented on PR #14191: URL: https://github.com/apache/kafka/pull/14191#issuecomment-1675305377
A few extra thoughts: - We may be unearthing fundamental limitations with the static `Exit` API if we can't cleanly/easily force testing-only clusters to not invoke `System::exit` (or similar methods). It may be more appropriate to develop a non-static API similar to the [Time interface](https://github.com/apache/kafka/blob/f5655d31d3d527dae057240570162827c6a79fb2/clients/src/main/java/org/apache/kafka/common/utils/Time.java) so that we can have complete control over our testing clusters - Since https://github.com/apache/kafka/pull/14156 was merged, there have been no test failures on trunk for the `testOffsetTranslationBehindReplicationFlow` case. We can look into the other failing tests, but this one was the highest-priority because we had reason to believe that it was indicative of legitimate issues with MM2 (and these suspicions proved correct) -- 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
