[ 
https://issues.apache.org/jira/browse/IGNITE-3014?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Semen Boikov reassigned IGNITE-3014:
------------------------------------

    Assignee: Ilya Lantukh  (was: Semen Boikov)

Hi, reviewed, my comments:
- I think we need keep readLock in localPartitionMap() (otherwise topVer, 
updateSeq can change, it is better leave it as is)
- please review usages of 'localPartitions()', is it possible to use 
'currentLocalPartitions()' everywhere?
- remove unused imports in GridDhtPartitionTopologyImpl

> Optimize GridDhtPartitionTopologyImpl#localPartition()
> ------------------------------------------------------
>
>                 Key: IGNITE-3014
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3014
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Ilya Lantukh
>            Assignee: Ilya Lantukh
>              Labels: performance
>
> This method is called at least once for every cache operation on each node.
> It was partially optimized in 
> https://issues.apache.org/jira/browse/IGNITE-2948.
> It seems that we can reduce time spent in that method even further by 
> removing excessive RW locks and using volatile read/write instead.



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

Reply via email to