[ 
https://issues.apache.org/jira/browse/MINIFI-207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16023126#comment-16023126
 ] 

ASF subversion and git services commented on MINIFI-207:
--------------------------------------------------------

Commit cbc12903ffb81bf16808f61a42e772d7c4676c06 in nifi-minifi-cpp's branch 
refs/heads/master from Marc Parisi
[ https://git-wip-us.apache.org/repos/asf?p=nifi-minifi-cpp.git;h=cbc1290 ]

MINIFI-207: Use recursive mutex that avoid thread safety concerns

This closes #102.

Signed-off-by: Aldrin Piri <ald...@apache.org>


> findProcessor in ProcessGroup not thread safe
> ---------------------------------------------
>
>                 Key: MINIFI-207
>                 URL: https://issues.apache.org/jira/browse/MINIFI-207
>             Project: Apache NiFi MiNiFi
>          Issue Type: Bug
>          Components: C++
>    Affects Versions: cpp-0.1.0
>            Reporter: marco polo
>            Assignee: marco polo
>             Fix For: cpp-0.3.0
>
>
> These functions do not use the mutex. the sets can be change the underlying 
> tree upon insertion. Since flow cannot be guaranteed we should use the mutex 
> here ( i.e. it can be called in any order since it is public ). 
> add and remove connection call this function and lock the mutex themselves. 
> An alternative might be to use a recursive mutex or dramatically reduce the 
> critical section in add/remove connection. This appears more desirable. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to