You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Mikhail Petrov (Jira)" <ji...@apache.org> on 2020/04/20 09:12:00 UTC
[jira] [Commented] (IGNITE-12907) GridSystemViewManager is not
thread safe
[ https://issues.apache.org/jira/browse/IGNITE-12907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087519#comment-17087519 ]
Mikhail Petrov commented on IGNITE-12907:
-----------------------------------------
[~agura], could you clarify, please. I see no {{registerWalker()}} method in GridSystemViewManager and ConcurrentHashMap is used to store the SystemView at current master. Does this mean that the problem has been already solved?
> GridSystemViewManager is not thread safe
> ----------------------------------------
>
> Key: IGNITE-12907
> URL: https://issues.apache.org/jira/browse/IGNITE-12907
> Project: Ignite
> Issue Type: Bug
> Reporter: Andrey N. Gura
> Assignee: Nikolay Izhikov
> Priority: Major
> Labels: IEP-35
> Fix For: 2.8.1
>
>
> {{GridSystemViewManager}} is not thread safe because it allows to registers walkers using public {{registerWalker()}} that just adds walker to a {{HashMap}}.
> It seems the simplest solution here: make {{registerWalker()}} method private because it has usages only from {{GridSystemViewManager}} constructor.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)