squah-confluent opened a new pull request, #15559:
URL: https://github.com/apache/kafka/pull/15559
KIP-890 Part 1 introduced verification of transactions with the transaction
coordinator on the `Produce` and `TxnOffsetCommit` paths. This introduced the
possibility of new errors when responding to those requests. For backwards
compatibility with older clients, a choice was made to convert some of the new
retriable errors to existing errors that are expected and retried correctly by
older clients.
`NETWORK_EXCEPTION` was forgotten about and not converted, but can occur if,
for example, the transaction coordinator is temporarily refusing connections.
Now, we convert it to:
* `NOT_ENOUGH_REPLICAS` on the `Produce` path, just like the other
retriable errors that can arise from transaction verification.
* `COORDINATOR_LOAD_IN_PROGRESS` on the `TxnOffsetCommit` path. This error
does not force coordinator lookup on clients, unlike
`COORDINATOR_NOT_AVAILABLE`. Note that this deviates from KIP-890, which says
that retriable errors should be converted to `COORDINATOR_NOT_AVAILABLE`.
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgrade notes)
--
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
