[jira] [Commented] (IGNITE-2508) GridDhtPartitionTopologyImpl#nodes(int, AffinityTopologyVersion) call are too costly.

["Alexey Goncharuk (JIRA)" <ji...@apache.org>]

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

Alexey Goncharuk commented on IGNITE-2508:
------------------------------------------

Vladimir,

Changes look ok to me. I do not think that caching a set of UUIDs can significantly increase memory footprint. As for the other potential places, nothing pops into my head right away. I will create a ticket if I remember something.

> GridDhtPartitionTopologyImpl#nodes(int, AffinityTopologyVersion) call are too costly.
> -------------------------------------------------------------------------------------
>
>                 Key: IGNITE-2508
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2508
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: general
>    Affects Versions: 1.5.0.final
>            Reporter: Ilya Lantukh
>            Assignee: Alexey Goncharuk
>             Fix For: 1.6
>
>
> New HashSet and ArrayList are instantiated on every call. Check if it is possible to optimize subsequent calls of this method if affinity assignment hasn't changed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)