wcarlson5 commented on a change in pull request #9487:
URL: https://github.com/apache/kafka/pull/9487#discussion_r523334732



##########
File path: 
streams/src/test/java/org/apache/kafka/streams/integration/KStreamRepartitionIntegrationTest.java
##########
@@ -144,6 +143,7 @@ public void whenShuttingDown() throws IOException {
     }
 
     @Test
+    @Deprecated //a single thread should no longer die

Review comment:
       I agree we shouldn't remove the valid test cases. Maybe the ones that 
are more complicated I can just set an idempotent old handler and mark as 
deprecated and we can file tickets to update. Either we work them down or when 
we go to remove the old handler they will fail and we need to fix them then.




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


Reply via email to