nicktelford commented on PR #16922: URL: https://github.com/apache/kafka/pull/16922#issuecomment-2331263893
@mjsax Following our conversation about suspend/resume of `StandbyTask` on Slack, I've changed the implementation to no longer `suspend` the pending tasks after initializing them (see https://github.com/apache/kafka/pull/16922/commits/4c07d8facd12b84fff40c5e1b28f1b94ff4169e9). This means they will technically be in the `RUNNING` state before they've been assigned, but they won't actually be executed, because they won't have been added to a `TaskRegistry` or `StateUpdater`. My thinking was that if `SUSPENDED` gets removed in the future, it would make no sense to depend on it here, and if we used that state, we would need to `resume` some StandbyTasks after assignment, which currently is not possible. -- 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
