Should we move this into the and `else` branch of `if (commitTimeMs >= 0 && lastCommitMs + commitTimeMs < now)` -- if the condition is true, we call `taskManager.commitAll()` and thus `taskManager.maybeCommitActiveTasks()` seems to be redundant here?
[ Full content available at: https://github.com/apache/kafka/pull/5428 ] This message was relayed via gitbox.apache.org for [email protected]
