You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Kirk Lund (JIRA)" <ji...@apache.org> on 2018/02/01 18:21:00 UTC
[jira] [Updated] (GEODE-3583) Remove all usages of static methods
to the Cache
[ https://issues.apache.org/jira/browse/GEODE-3583?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kirk Lund updated GEODE-3583:
-----------------------------
Description:
This ticket is the parent ticket to remove all invocations of Cache.getAnyInstance(), GemFireCacheImpl.getInstance(), etc...
This improvement is complete is there are no more usages of the static methods to retrieve a Cache
As per some work that [~huynhja] has done
_ grep -rl "GemFireCacheImpl.getInstance" --include \*.java . > GemFireCacheImplList.txt
grep -rl "CacheFactory.getAnyInstance" --include \*.java . > CacheFactoryList.txt
grep -rl "InternalDistributedSystem.getAnyInstance" --include \*.java . > InternalDistributedSystemList.txt
I've attached the files to this email...
The combinedList.txt is a grep that combined all 3 calls into a single file... _
[EDIT] Additional singleton getters that need to be removed:
* CacheFactory.getInstance(DistributedSystem)
* CacheFactory.getInstanceCloseOk(DistributedSystem)
* GemFireCacheImpl.getExisting()
* GemFireCacheImpl.getExisting(String)
was:
This ticket is the parent ticket to remove all invocations of Cache.getAnyInstance(), GemFireCacheImpl.getInstance(), etc...
This improvement is complete is there are no more usages of the static methods to retrieve a Cache
As per some work that [~huynhja] has done
_ grep -rl "GemFireCacheImpl.getInstance" --include \*.java . > GemFireCacheImplList.txt
grep -rl "CacheFactory.getAnyInstance" --include \*.java . > CacheFactoryList.txt
grep -rl "InternalDistributedSystem.getAnyInstance" --include \*.java . > InternalDistributedSystemList.txt
I've attached the files to this email...
The combinedList.txt is a grep that combined all 3 calls into a single file... _
> Remove all usages of static methods to the Cache
> ------------------------------------------------
>
> Key: GEODE-3583
> URL: https://issues.apache.org/jira/browse/GEODE-3583
> Project: Geode
> Issue Type: Improvement
> Reporter: Udo Kohlmeyer
> Priority: Major
> Labels: pull-request-available
> Attachments: CacheFactoryList.txt, CombinedList.txt, GemFireCacheImplList.txt, InternalDistributedSystemList.txt
>
> Time Spent: 1h
> Remaining Estimate: 0h
>
> This ticket is the parent ticket to remove all invocations of Cache.getAnyInstance(), GemFireCacheImpl.getInstance(), etc...
> This improvement is complete is there are no more usages of the static methods to retrieve a Cache
> As per some work that [~huynhja] has done
> _ grep -rl "GemFireCacheImpl.getInstance" --include \*.java . > GemFireCacheImplList.txt
> grep -rl "CacheFactory.getAnyInstance" --include \*.java . > CacheFactoryList.txt
> grep -rl "InternalDistributedSystem.getAnyInstance" --include \*.java . > InternalDistributedSystemList.txt
> I've attached the files to this email...
> The combinedList.txt is a grep that combined all 3 calls into a single file... _
> [EDIT] Additional singleton getters that need to be removed:
> * CacheFactory.getInstance(DistributedSystem)
> * CacheFactory.getInstanceCloseOk(DistributedSystem)
> * GemFireCacheImpl.getExisting()
> * GemFireCacheImpl.getExisting(String)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)