mjsax commented on code in PR #19507: URL: https://github.com/apache/kafka/pull/19507#discussion_r2051247959
########## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ########## @@ -553,7 +553,7 @@ private void handleTasksWithStateUpdater(final Map<TaskId, Set<TopicPartition>> private void handleTasksPendingInitialization() { // All tasks pending initialization are not part of the usual bookkeeping for (final Task task : tasks.drainPendingTasksToInit()) { - closeTaskClean(task, Collections.emptySet(), Collections.emptyMap()); + closeTaskClean(task, new HashSet<>(), new HashMap<>()); Review Comment: Thanks @FrankYang0529 -- still not sure right now if just reverting is the right thing: > that is a good question ... The is the thing that worries me -- we did see it crash with EOSv2, so some pending tasks did get revoked and there was an error during revocation. So maybe we did expose to different bug. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org