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/21 01:51:32 UTC

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #14745: [DOC] pulsar web service executor metrics doc

Anonymitaet commented on a change in pull request #14745:
URL: https://github.com/apache/pulsar/pull/14745#discussion_r830711144



##########
File path: site2/docs/reference-metrics.md
##########
@@ -495,6 +496,22 @@ All the jetty metrics are labelled with the following labels:
 | jetty_stats_seconds | Gauge | Time in seconds stats have been collected for. |
 | jetty_responses_bytes_total | Counter | Total number of bytes across all responses. |
 
+### Web service executor metrics
+
+> For a functions-worker running separately from brokers, its Jetty metrics are only exposed when `includeStandardPrometheusMetrics` is set to `true`.

Review comment:
       ```suggestion
   > For functions workers running separately from brokers, their Jetty metrics are only exposed when `includeStandardPrometheusMetrics` is set to `true`.
   ```

##########
File path: site2/docs/reference-metrics.md
##########
@@ -495,6 +496,22 @@ All the jetty metrics are labelled with the following labels:
 | jetty_stats_seconds | Gauge | Time in seconds stats have been collected for. |
 | jetty_responses_bytes_total | Counter | Total number of bytes across all responses. |
 
+### Web service executor metrics
+
+> For a functions-worker running separately from brokers, its Jetty metrics are only exposed when `includeStandardPrometheusMetrics` is set to `true`.
+
+All the web service executor metrics are labelled with the following labels:
+
+- *cluster*: `cluster=${pulsar_cluster}`. `${pulsar_cluster}` is the cluster name that you have configured in the `broker.conf` file.
+
+| 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 |

Review comment:
       the min threads

##########
File path: site2/docs/reference-metrics.md
##########
@@ -495,6 +496,22 @@ All the jetty metrics are labelled with the following labels:
 | jetty_stats_seconds | Gauge | Time in seconds stats have been collected for. |
 | jetty_responses_bytes_total | Counter | Total number of bytes across all responses. |
 
+### Web service executor metrics
+
+> For a functions-worker running separately from brokers, its Jetty metrics are only exposed when `includeStandardPrometheusMetrics` is set to `true`.
+
+All the web service executor metrics are labelled with the following labels:
+
+- *cluster*: `cluster=${pulsar_cluster}`. `${pulsar_cluster}` is the cluster name that you have configured in the `broker.conf` file.
+
+| 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 | How many threads doing task of pulsar-web thread pool |

Review comment:
       ```suggestion
   | pulsar_web_executor_active_threads | GAUGE | The number of threads performing tasks of pulsar-web thread pool |
   ```

##########
File path: site2/docs/reference-metrics.md
##########
@@ -495,6 +496,22 @@ All the jetty metrics are labelled with the following labels:
 | jetty_stats_seconds | Gauge | Time in seconds stats have been collected for. |
 | jetty_responses_bytes_total | Counter | Total number of bytes across all responses. |
 
+### Web service executor metrics
+
+> For a functions-worker running separately from brokers, its Jetty metrics are only exposed when `includeStandardPrometheusMetrics` is set to `true`.
+
+All the web service executor metrics are labelled with the following labels:
+
+- *cluster*: `cluster=${pulsar_cluster}`. `${pulsar_cluster}` is the cluster name that you have configured in the `broker.conf` file.
+
+| 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 | How many threads doing task of pulsar-web thread pool |
+| pulsar_web_executor_current_threads | GAUGE | How many threads in the pulsar-web thread pool now |

Review comment:
       ```suggestion
   | pulsar_web_executor_current_threads | GAUGE | The number of threads in the pulsar-web thread pool |
   ```




-- 
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