ableegoldman commented on a change in pull request #10417:
URL: https://github.com/apache/kafka/pull/10417#discussion_r602616135
##########
File path:
streams/src/test/java/org/apache/kafka/streams/integration/StreamsUncaughtExceptionHandlerIntegrationTest.java
##########
@@ -177,6 +178,7 @@ public void shouldShutdownMultipleThreadApplication()
throws InterruptedExceptio
testShutdownApplication(2);
}
+ @Ignore // TODO -- re-enable this test after KAFKA-12537
Review comment:
I suspect whichever one is merged first will result in small conflicts
on the other since they both touch `StreamThread`. But maybe you can remove the
changes in StreamThread from your PR since I think they're encompasses by this
one, and I'll remove the `Ignore` so they can both be merged once the build
passes -- WDYT?
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
