[ 
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 
CacheFactory.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] Complete list of all singleton getters for which calls need to be 
removed:
* CacheFactory.getAnyInstance()
* CacheFactory.getInstance(DistributedSystem)
* CacheFactory.getInstanceCloseOk(DistributedSystem)
* GemFireCacheImpl.getExisting()
* GemFireCacheImpl.getExisting(String)
* GemFireCacheImpl.getForPdx(String)
* GemFireCacheImpl.getInstance()
* GemFireCacheImpl.setInstanceForTests(GemFireCacheImpl)
* InternalDistributedSystem.getAnyInstance()
* InternalDistributedSystem getConnectedInstance()
* InternalDistributedSystem unsafeGetConnectedInstance()


  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... _

[EDIT] Additional singleton getters that need to be removed:
* CacheFactory.getInstance(DistributedSystem)
* CacheFactory.getInstanceCloseOk(DistributedSystem)
* GemFireCacheImpl.getExisting()
* GemFireCacheImpl.getExisting(String)
* GemFireCacheImpl.getForPdx(String)
* GemFireCacheImpl.setInstanceForTests(GemFireCacheImpl)
* InternalDistributedSystem.getAnyInstance()
* InternalDistributedSystem getConnectedInstance()
* InternalDistributedSystem unsafeGetConnectedInstance()



> 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 
> CacheFactory.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] Complete list of all singleton getters for which calls need to be 
> removed:
> * CacheFactory.getAnyInstance()
> * CacheFactory.getInstance(DistributedSystem)
> * CacheFactory.getInstanceCloseOk(DistributedSystem)
> * GemFireCacheImpl.getExisting()
> * GemFireCacheImpl.getExisting(String)
> * GemFireCacheImpl.getForPdx(String)
> * GemFireCacheImpl.getInstance()
> * GemFireCacheImpl.setInstanceForTests(GemFireCacheImpl)
> * InternalDistributedSystem.getAnyInstance()
> * InternalDistributedSystem getConnectedInstance()
> * InternalDistributedSystem unsafeGetConnectedInstance()



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to