AndrewJSchofield commented on code in PR #22684: URL: https://github.com/apache/kafka/pull/22684#discussion_r3639501873
########## docs/operations/monitoring.md: ########## @@ -3447,44 +3447,45 @@ The total time the Consumer spent committing offsets in nanoseconds (for AOS). <td> kafka.consumer:type=consumer-metrics,client-id=([-.\w]+) -</td> </tr> +</td> </tr> <tr> <td> -time-between-network-thread-poll</td> +application-event-queue-size +</td> <td> -The average delay between invocations of poll (for CONSUMER protocol) +The current number of events in the queue to send from the application thread to the background thread(for CONSUMER Protocol,will be referred as 'CP' in the below metrics). Review Comment: nit: Spacing and a few words can be improved. `background thread (for CONSUMER protocol, will be referred to as 'CP' in the following metrics)` ########## docs/operations/monitoring.md: ########## @@ -3493,11 +3494,24 @@ application-event-queue-processing-time-avg </td> <td> -The maximum time, in milliseconds, that the consumer network took to process all available application events (for CONSUMER protocol) +The average time, in ms,that the background thread took to process all available application events((for CP only). Review Comment: nit: `events((forCP` should be `events (for CP`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
