You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/02/24 16:14:13 UTC

[GitHub] [pulsar] frankjkelly opened a new issue #14459: [Broker] Expose prometheus metric for current and max number of http server threads

frankjkelly opened a new issue #14459:
URL: https://github.com/apache/pulsar/issues/14459


   **Is your enhancement request related to a problem? Please describe.**
   I'd like to monitor to ensure that Jetty has enough server threads to handle the number of incoming requests.
   
   As discussed by @lhotari here https://github.com/apache/pulsar/issues/14329
   "Since Pulsar Admin API uses the blocking servlet API, all Jetty threads might be occupied and this causes unavailability of the Pulsar Admin API. "
   
   **Describe the solution you'd like**
   Expose Prometheus metrics for the following
   1) Max configured size of the Jetty thread pool (I assume it's a pool?)
   2) Current number of threads in use
   
   **Describe alternatives you've considered**
   Logging of number of threads is not very actionable.
   
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] tjiuming commented on issue #14459: [Broker] Expose prometheus metric for current and max number of http server threads

Posted by GitBox <gi...@apache.org>.
tjiuming commented on issue #14459:
URL: https://github.com/apache/pulsar/issues/14459#issuecomment-1067623918


   @codelipenghui  I'd like to handle the issue, please assign to me


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] tjiuming commented on issue #14459: [Broker] Expose prometheus metric for current and max number of http server threads

Posted by GitBox <gi...@apache.org>.
tjiuming commented on issue #14459:
URL: https://github.com/apache/pulsar/issues/14459#issuecomment-1084831240


   @frankjkelly Hi frank, I had created the pr (https://github.com/apache/pulsar/pull/14742) weeks ago and it's in reviewing now


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] frankjkelly commented on issue #14459: [Broker] Expose prometheus metric for current and max number of http server threads

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #14459:
URL: https://github.com/apache/pulsar/issues/14459#issuecomment-1084836179


   Great - thanks @tjiuming. @sijie is it possible to get some eyes on that PR and then maybe we could seek to backport it to some prior maintenance releases?


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] frankjkelly commented on issue #14459: [Broker] Expose prometheus metric for current and max number of http server threads

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #14459:
URL: https://github.com/apache/pulsar/issues/14459#issuecomment-1050020557


   @sijie here is the enhancement request we discussed yesterday 


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] frankjkelly commented on issue #14459: [Broker] Expose prometheus metric for current and max number of http server threads

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #14459:
URL: https://github.com/apache/pulsar/issues/14459#issuecomment-1084777377


   Hello @tjiuming have you been able to make any progress on this? Thanks! CC: @sijie 
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org