lucasbru commented on code in PR #14555:
URL: https://github.com/apache/kafka/pull/14555#discussion_r1360674675
##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/Tasks.java:
##########
@@ -240,8 +241,8 @@ public synchronized void addTask(final Task task) {
public synchronized void removeTask(final Task taskToRemove) {
final TaskId taskId = taskToRemove.id();
- if (taskToRemove.state() != Task.State.CLOSED) {
- throw new IllegalStateException("Attempted to remove a task that
is not closed: " + taskId);
+ if (taskToRemove.state() != Task.State.CLOSED && taskToRemove.state()
!= State.SUSPENDED) {
Review Comment:
:( I actually wanted to move the pause after the removal but forgot it. Good
catch!
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
