dajac commented on PR #13765:
URL: https://github.com/apache/kafka/pull/13765#issuecomment-1568017910

   > Yes, that is fair too. The definition of leadership epoch in which case 
changes to - it represents the version of the leader after a re-election. In 
this case, we should also remove the epoch change during ISR expansion as well. 
My point is, let's keep the definition as state of ISR (current) or state of 
leader (in which case we remove epoch change for both expansion and shrink).
   
   Yeah, I agree that we need to do both in order to remain consistent.
   
   > Aside for it, out of curiosity, is there any other version which 
represents the state of ISR in Kafka? Does replica epoch changes on every 
change to ISR?
   
   There is the partition epoch which is incremented whenever the partition is 
updated. This includes ISR changes.


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

Reply via email to