ableegoldman commented on a change in pull request #8900:
URL: https://github.com/apache/kafka/pull/8900#discussion_r443950906



##########
File path: 
streams/src/test/java/org/apache/kafka/streams/processor/internals/RecordCollectorTest.java
##########
@@ -474,6 +492,7 @@ public void 
shouldThrowTaskMigratedExceptionOnSubsequentCallWhenProducerFencedIn
                 " indicating the task may be migrated out; it means all tasks 
belonging to this thread should be migrated.")
         );
 
+        collector.send(topic, "3", "0", null, null, stringSerializer, 
stringSerializer, streamPartitioner);

Review comment:
       These tests used to rely on the fact that the `sendException` was never 
forgotten by just setting it once and then asserting that multiple subsequent 
calls also threw it. So now we need to call `send` before each to re-insert the 
exception




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