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