artemlivshits commented on PR #17402: URL: https://github.com/apache/kafka/pull/17402#issuecomment-2403547541
> This would only be if TV2 is used right? It might not be as simple as removing everything. Yes, basically do `if (!v2Enabled) epochBumpRequired = true` in the 3 cases where it's done for transactional producer. +add a comment explaining `epochBumpRequired` logic because now it has 3 cases: idempotent producers, transactional v1 (need to bump epoch from the client to fence off zombies from the transaction to be aborted), v2 bumps epoch automatically, so zombies will get fenced without explicit bump from the client. -- 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
