Hi Roger, Thank you for the review. I have added a table with the scope of errors counted for each request.
Regards, Rajini On Thu, Aug 17, 2017 at 12:05 AM, Roger Hoover <roger.hoo...@gmail.com> wrote: > I think it would useful to make clear somewhere for each metric, the level > at which it's counted. I don't know all the details of the Kafka protocol > but it might be something like > > ProduceRequest, Fetch Request - counted at per-partition level > All other requests are 1:1 with client requests? > > Cheers, > > Roger > > On Wed, Aug 16, 2017 at 4:02 PM, Roger Hoover <roger.hoo...@gmail.com> > wrote: > > > Rajini, > > > > Thank you for the KIP. These are very helpful additions. One question > on > > the error code metrics: > > > > Will the total error counting happen at the the level of topic partition? > > For example, if a single ProduceRequest contains messages to append to 3 > > partitions and say all 3 appends are successful, the counter > > for kafka.network:type=RequestMetrics,name=ErrorsPerSec,request= > ProduceRequest,error=0 > > will be incremented by 3? > > > > Thanks, > > > > Roger > > > > On Wed, Aug 16, 2017 at 12:10 PM, Rajini Sivaram < > rajinisiva...@gmail.com> > > wrote: > > > >> I have created a KIP to add some additional metrics to support health > >> checks: > >> > >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-188+-+ > >> Add+new+metrics+to+support+health+checks > >> > >> Feedback and suggestions are welcome. > >> > >> Regards, > >> > >> Rajini > >> > > > > >
