[
https://issues.apache.org/jira/browse/DISPATCH-1278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16785753#comment-16785753
]
ASF GitHub Bot commented on DISPATCH-1278:
------------------------------------------
ted-ross commented on pull request #453: DISPATCH-1278: initial support for
prometheus metrics export
URL: https://github.com/apache/qpid-dispatch/pull/453#discussion_r262996925
##########
File path: src/http-libwebsockets.c
##########
@@ -361,6 +386,135 @@ static void connection_wake(qd_connection_t *qd_conn)
}
}
+static void handle_stats_results(void *context, qdr_global_stats_t *results)
+{
+ stats_t* stats = (stats_t*) context;
+ qd_http_server_t *hs = stats->server;
+ if (hs) {
+ work_t w = { W_HANDLE_STATS, stats->wsi };
+ work_push(hs, w);
Review comment:
It should be noted that this handler is being called on the core-thread.
Since work_push can block on a condition, the core thread is at risk for being
blocked by the HTTP processing. It might be better for the core-thread to
schedule this callback as a general-work item so it runs on an I/O thread.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
> Add support for prometheus metrics export
> -----------------------------------------
>
> Key: DISPATCH-1278
> URL: https://issues.apache.org/jira/browse/DISPATCH-1278
> Project: Qpid Dispatch
> Issue Type: New Feature
> Reporter: Gordon Sim
> Assignee: Gordon Sim
> Priority: Major
> Fix For: 1.6.0
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
