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é

Reply via email to