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/03/18 08:07:52 UTC

[GitHub] [pulsar] tjiuming opened a new pull request #14742: [Broker] pulsar web executor metrics

tjiuming opened a new pull request #14742:
URL: https://github.com/apache/pulsar/pull/14742


   Fixes https://github.com/apache/pulsar/issues/14459
   
   ### Motivation
   
   See the issue
   
   ### Modifications
   1. Add WebExecutorStats to record web thread pool metrics
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (no)
     - The public API: (no)
     - The schema: (no)
     - The default values of configurations: (no)
     - The wire protocol: (no)
     - The rest endpoints: (no)
     - The admin cli options: (no)
     - Anything that affects deployment: (no)
   
   ### Documentation
   
   Check the box below or label this PR directly (if you have committer privilege).
   
   Need to update docs? 
   
   - [x] `doc-required` 
     
     (If you need help on updating docs, create a doc issue)
     
   - [ ] `no-need-doc` 
     
     (Please explain why)
     
   - [ ] `doc` 
     
     (If this PR contains doc changes)
   
   
   


-- 
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] codelipenghui commented on pull request #14742: [Broker] pulsar web executor metrics

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on pull request #14742:
URL: https://github.com/apache/pulsar/pull/14742#issuecomment-1072136370


   @tjiuming Thanks for the great contribution, this PR will bring more transparency to the web service thread pool. Please help open another PR to add the documentation.


-- 
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 pull request #14742: [Broker] pulsar web executor metrics

Posted by GitBox <gi...@apache.org>.
tjiuming commented on pull request #14742:
URL: https://github.com/apache/pulsar/pull/14742#issuecomment-1072126275


   this pr add following metrics:
   |name|type|description|
   |--|--|--|
   |pulsar_web_executor_max_threads|GAUGE|The max threads of pulsar-web  thread pool|
   |pulsar_web_executor_min_threads|GAUGE|The max threads of pulsar-web thread pool|
   |pulsar_web_executor_idle_threads|GAUGE|The idle threads of pulsar-web thread pool|
   |pulsar_web_executor_active_threads|GAUGE|The active-threads(doing task) of pulsar-web thread pool|
   |pulsar_web_executor_current_threads|GAUGE|How many threads in the pulsar-web thread pool 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] codelipenghui commented on pull request #14742: [Broker] pulsar web executor metrics

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on pull request #14742:
URL: https://github.com/apache/pulsar/pull/14742#issuecomment-1072136370


   @tjiuming Thanks for the great contribution, this PR will bring more transparency to the web service thread pool. Please help open another PR to add the documentation.


-- 
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 pull request #14742: [Broker] pulsar web executor metrics

Posted by GitBox <gi...@apache.org>.
tjiuming commented on pull request #14742:
URL: https://github.com/apache/pulsar/pull/14742#issuecomment-1072126275


   this pr add following metrics:
   |name|type|description|
   |--|--|--|
   |pulsar_web_executor_max_threads|GAUGE|The max threads of pulsar-web  thread pool|
   |pulsar_web_executor_min_threads|GAUGE|The max threads of pulsar-web thread pool|
   |pulsar_web_executor_idle_threads|GAUGE|The idle threads of pulsar-web thread pool|
   |pulsar_web_executor_active_threads|GAUGE|The active-threads(doing task) of pulsar-web thread pool|
   |pulsar_web_executor_current_threads|GAUGE|How many threads in the pulsar-web thread pool 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