hachikuji commented on code in PR #12071:
URL: https://github.com/apache/kafka/pull/12071#discussion_r855395294
##########
clients/src/main/resources/common/message/LeaderAndIsrRequest.json:
##########
@@ -78,8 +78,8 @@
"about": "The leader epoch." },
{ "name": "Isr", "type": "[]int32", "versions": "0+", "entityType":
"brokerId",
"about": "The in-sync replica IDs." },
- { "name": "ZkVersion", "type": "int32", "versions": "0+",
- "about": "The ZooKeeper version." },
+ { "name": "PartitionEpoch", "type": "int32", "versions": "0+",
+ "about": "The current epoch for the partition for KRaft controllers.
The current ZK version for the legacy controllers." },
Review Comment:
nit: It does seem a little misleading to mention KRaft here since the
`LeaderAndIsr` API is not used. Maybe we could somehow call that out?
More generally, do you know if there is a good place we can document
assumptions about the partition epoch? Mainly looking for a good place we can
document that it is a monotonically increasing value which is incremented after
every partition change. Maybe `kafka.api.LeaderAndIsr`?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
