[
https://issues.apache.org/jira/browse/GEODE-4196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16319403#comment-16319403
]
Darrel Schneider commented on GEODE-4196:
-----------------------------------------
org.apache.geode.cache.CacheClosedException.CacheClosedException(String) is
calling getInstance to conditionally set the cause of the exception.
I think we should just change this constructor to no longer do this and have
callers do one of the following:
1. if they have access to a cache instance then call
getCacheClosedException(msg, null) on it. If that cache has a reason for being
disconnected then it will set the cause.
2. if they do not have access to a cache then it is probably just best to call
CacheClosedException(msg) with no cause. If they do have a call then they can
call CacheClosedException(String, Throwable)
> CacheClosedException should not invoke GemFireCacheImpl.getInstance
> -------------------------------------------------------------------
>
> Key: GEODE-4196
> URL: https://issues.apache.org/jira/browse/GEODE-4196
> Project: Geode
> Issue Type: Sub-task
> Components: regions
> Reporter: Kirk Lund
> Assignee: Kirk Lund
>
> CacheClosedException should not invoke GemFireCacheImpl.getInstance
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
