dino2895 opened a new pull request, #23250: URL: https://github.com/apache/kafka/pull/23250
Follow-up to https://github.com/apache/kafka/pull/19528#discussion_r3832670432 The original Scala test asserted on the metric values observed by the background thread. After the port, the assertions re-read the gauges at the end instead. Since the background thread stops as soon as it sees a deviation, and the gauges settle back once the topics are deleted, that deviation is discarded and the test still passes. Store each sample in an `AtomicInteger` and assert on those, restoring the original semantics. Also re-set the interrupt flag on `InterruptedException` instead of ignoring it. ### Testing `./gradlew :clients:clients-integration-tests:test --tests '*MetricsDuringTopicCreationDeletionTest*'` `./gradlew :clients:clients-integration-tests:checkstyleTest` -- 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]
