Hi Jun, On Wed, Apr 30, 2025 at 1:09 PM Jun Rao <j...@confluent.io.invalid> wrote: > JR5. The motivation section still sounds like this is a KRaft only problem. > It would be useful to cover the issue with fetch from followers too.
Okay. I updated the motivation section to also talk about fetch-from-followers. I was hesitant to add this to the motivation since I didn't have concrete data for this motivation but it makes sense in theory. I also updated the linked jira issue to now track two tasks. One for updating the kraft implementation and one for updating the replica manager implementation. > JR6. Since there is an inter broker RPC change, we need to document the > upgrade path with a new MV. Good point. We version inter-broker FETCH requests using the metadata version. I added that detail to the KIP. > JR7. Could we document whether the consumer client should set the new field > or not? In short, Kafka consumers should not set the field and the receiving replicas will ignore the field if the fetching client is a consumer. I updated the KIP with this information. Thanks, -- -José