You are correct Raman. I updated the KIP to reflect your observations
and corrections.

Raman wrote:
> I think the line #2 here is wrong with regard to AlterPartitionRequest.

Yes. Bullet 2. Now reads: 2. The LeaderRecoveryState is changing from
RECOVERED to RECOVERING.

Raman wrote:
> Also, in the response schema, I think the last line is a bit
> misleading. As we discussed offline, the field can be either
> Recovering or Recovered, depending on when the leader sent out an
> AlterPartition request to the controller (i.e. during recovery or
> after recovery has completed). But the broker does not take the value
> in this field as a trigger to start recovery. That happens through
> either the LeaderAndIsr request or the relevant change record in Kraft
> world.

Yes. I updated the KIP to make this clear.

Thanks!
-- 
-José

Reply via email to