philipnee commented on code in PR #12149:
URL: https://github.com/apache/kafka/pull/12149#discussion_r1171904037
##########
clients/src/main/java/org/apache/kafka/clients/producer/internals/TransactionManager.java:
##########
@@ -1294,7 +1309,10 @@ public void handleResponse(AbstractResponse response) {
reenqueue();
} else if (error == Errors.TRANSACTIONAL_ID_AUTHORIZATION_FAILED ||
error == Errors.CLUSTER_AUTHORIZATION_FAILED) {
- fatalError(error.exception());
+ log.info("Abortable authorization error: {}. Transition the
producer state to {}", error.message(), State.ABORTABLE_ERROR);
+ lastError = error.exception();
+ epochBumpRequired = true;
Review Comment:
@jolshan - See the last commit. I cleaned up the comments because
authorization isn't a fatal error after the patch. I also patched a test to
demonstrate resend after fixing the permission should bring the producer back
to normal. Also tested the epoch there to ensure it is 0, not -1 after fixing
the permission.
--
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
