KAFKA-19998: KRaft client should expose the latest committed kraft version for some readers#21205
KAFKA-19998: KRaft client should expose the latest committed kraft version for some readers#21205kevin-wu24 wants to merge 2 commits intoapache:trunkfrom
Conversation
chia7712
left a comment
There was a problem hiding this comment.
@kevin-wu24 thanks for this patch. one dumb question: what are the drawbacks if API Versions response uses the "uncommitted" kraft version?
It's mainly a UX issue that impacts clusters undergoing a kraft version upgrade, or clusters who have not committed a "bootstrapped" kraft version 1 yet (i.e. because of network partitioning). The issue of exposing the uncommitted kraft version in the APIVersions response is that the finalized features level for kraft may be different depending on the node to which I send |
Readers like the
SimpleApiVersionsManagerand theMetadataCacheshould only know about the latest committed kraft version. Others, like the broker registration manager, need to know the potentially uncommitted kraft version.