You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2019/09/15 07:45:00 UTC
[jira] [Commented] (DISPATCH-1386) qdr_action_list_t could be
replaced by an array of struct q
[ https://issues.apache.org/jira/browse/DISPATCH-1386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929923#comment-16929923 ]
ASF GitHub Bot commented on DISPATCH-1386:
------------------------------------------
franz1981 commented on pull request #537: DISPATCH-1386 qdr_action_list_t could be replaced by an array of struct q
URL: https://github.com/apache/qpid-dispatch/pull/537#discussion_r324450180
##########
File path: src/router_core/router_core.c
##########
@@ -101,7 +188,11 @@ void qdr_core_free(qdr_core_t *core)
// Stop and join the thread
//
core->running = false;
- sys_cond_signal(core->action_cond);
+ sys_mutex_lock(core->action_lock);
+ if (core->sleeping) {
Review comment:
@kgiusti I forgot to mention that this PR contains an important optimization under load to reduce the signaling work of the worker threads ie to save signaling core if it isn't sleeping.
This is beneficial from many point of views although it won't show up on benchmarks: please let me know if I should move it into a separate PR/jira/commit :+1:
----------------------------------------------------------------
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:
users@infra.apache.org
> qdr_action_list_t could be replaced by an array of struct q
> -----------------------------------------------------------
>
> Key: DISPATCH-1386
> URL: https://issues.apache.org/jira/browse/DISPATCH-1386
> Project: Qpid Dispatch
> Issue Type: Improvement
> Components: Routing Engine
> Reporter: Francesco Nigro
> Priority: Major
>
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org