[
https://issues.apache.org/jira/browse/GEODE-3299?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16219359#comment-16219359
]
ASF GitHub Bot commented on GEODE-3299:
---------------------------------------
pdxrunner commented on a change in pull request #975: GEODE-3299: throw
CacheClosedException in FunctionContext.getCache()
URL: https://github.com/apache/geode/pull/975#discussion_r146962691
##########
File path:
geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ShowMissingDiskStoresFunction.java
##########
@@ -65,23 +65,25 @@ public void execute(FunctionContext context) {
}
}
}
+ } catch (CacheClosedException ignored) {
Review comment:
Precheckin ran green except for known issues from unrelated commit
44e8432e6637afbde478420c26e260630a922a5e
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Gfsh functions should acquire Cache from FunctionContext
> --------------------------------------------------------
>
> Key: GEODE-3299
> URL: https://issues.apache.org/jira/browse/GEODE-3299
> Project: Geode
> Issue Type: Bug
> Components: gfsh
> Reporter: Kirk Lund
> Assignee: Kenneth Howe
> Fix For: 1.4.0
>
>
> Gfsh functions in geode-core
> org.apache.geode.management.internal.cli.functions currently acquire a
> reference to the Cache in several ways including from a singleton in
> CacheFactory which is prone to deadlocking. They should be modified to
> uniformly acquire the Cache from FunctionContext instead.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
