[
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:
us...@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-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org
