Hi Calvin, Thanks for the improvement.
1. In the KIP, you suggest changing the Fetch request to "Rename the ReplicaId to BrokerId" and "Add a new Field BrokerEpoch". The Fetch RPC is used by replicas that are not brokers, for example controllers in KRaft. Can we keep the name "ReplicaId" and use "ReplicaEpoch". Both KRaft and ISR partitions have the concept of replica id and replica epoch but not necessarily the concept of a broker. 2. Since the new field "BrokerEpoch '' is ignorable, should it also have a default value? How about -1 since that is what you use in AlterPartittion RPC. -- -José
