You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2020/06/20 03:38:59 UTC

[GitHub] [kafka] zhaohaidao commented on pull request #8549: KAFKA-9911: Add new PRODUCER_FENCED error code

zhaohaidao commented on pull request #8549:
URL: https://github.com/apache/kafka/pull/8549#issuecomment-646932428


   > Resilience Improvement With Old Brokers
   When the client is on the latest version but the broker is old, the client shall still see INVALID_PRODUCER_EPOCH from produce response and transactional responses. In either case, we shall abort the current transaction and retry with transaction coordinator. If the transaction coordinator doesn't support InitProducerId with epoch, we should enter fatal state just as KIP-360.
   @abbccdda 
   This pr's treatment of INVALID_PRODUCER_EPOCH looks inconsistent with the description of kip-588 as quoted above.
   In this pr, INVALID_PRODUCER_EPOCH will be converted to PRODUCER_FENCED. It make senses as INVALID_PRODUCER_EPOCH from old broker should be treated as fatal error, the same as PRODUCER_FENCED.
   I'm not sure if the resilience improvement with old brokers is still needed.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org