Thanks for the comments, Dong. I've updated the KIP and addressed your 3 comments. Please take another look when you get a chance.
Lucas On Wed, Jun 13, 2018 at 2:00 PM, Dong Lin <lindon...@gmail.com> wrote: > Hey Luca, > > Thanks for the KIP. Looks good overall. Some comments below: > > - We usually specify the full mbean for the new metrics in the KIP. Can you > specify it in the Public Interface section similar to KIP-237 > <https://cwiki.apache.org/confluence/display/KAFKA/KIP- > 237%3A+More+Controller+Health+Metrics> > ? > > - Maybe we could follow the same pattern as KIP-153 > <https://cwiki.apache.org/confluence/display/KAFKA/KIP- > 153%3A+Include+only+client+traffic+in+BytesOutPerSec+metric>, > where we keep the existing sensor name "BytesInPerSec" and add a new sensor > "ReplicationBytesInPerSec", rather than replacing the sensor name " > BytesInPerSec" with e.g. "ClientBytesInPerSec". > > - It seems that the KIP changes the semantics of the broker config > "queued.max.requests" because the number of total requests queued in the > broker will be no longer bounded by "queued.max.requests". This probably > needs to be specified in the Public Interfaces section for discussion. > > > Thanks, > Dong > > > On Wed, Jun 13, 2018 at 12:45 PM, Lucas Wang <lucasatu...@gmail.com> > wrote: > > > Hi Kafka experts, > > > > I created KIP-291 to add a separate queue for controller requests: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-291% > > 3A+Have+separate+queues+for+control+requests+and+data+requests > > > > Can you please take a look and let me know your feedback? > > > > Thanks a lot for your time! > > Regards, > > Lucas > > >