Hi Rajini,

I've read through your KIP and it looks good, I only have two things to
1. How do we detect removed partitions in updatePartitionMetadata? I'm
presuming that the list here is the currently existing map of partitions,
so if something is removed it can be calculated as the diff of the current
and the previous update. Is that right?
2. PartitionMetadata contains only the leader at this moment, however there
are similar classes that contain more information, like the replicas, isr,
offline replicas. I think including them might make sense to provide a more
robust API. What do you think?


On Wed, Feb 21, 2018 at 7:57 PM, Rajini Sivaram <rajinisiva...@gmail.com>

> Hi all,
> I have submitted KIP-257 to enable customisation of client quota
> computation:
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> 257+-+Configurable+Quota+Management
> The KIP proposes to make quota management pluggable to enable group-based
> and partition-based quotas for clients.
> Feedback and suggestions are welcome.
> Thank you...
> Regards,
> Rajini

Reply via email to