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

Darrel Schneider commented on GEODE-4200:
-----------------------------------------

LocalRegion is using it for this method:
{noformat}
  static LocalRegion getRegionFromPath(DistributedSystem system, String path) {
    Cache cache = GemFireCacheImpl.getInstance();
    if (cache == null) {
      return null;
    } else {
      return (LocalRegion) cache.getRegion(path);
    }
  }
{noformat}
Note that we pass in a DistributedSystem even though this method does not use 
it.
All the callers of this method have a DistributionManager and you can call 
getCache on it.
So I think we should change this method to take an InternalCache instead of 
DistributedSystem.


> LocalRegion should not invoke GemFireCacheImpl.getInstance
> ----------------------------------------------------------
>
>                 Key: GEODE-4200
>                 URL: https://issues.apache.org/jira/browse/GEODE-4200
>             Project: Geode
>          Issue Type: Sub-task
>          Components: regions
>            Reporter: Kirk Lund
>            Assignee: Kirk Lund
>
> LocalRegion should not invoke GemFireCacheImpl.getInstance



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to