geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (GEODE-3299) Gfsh functions should acquire Cache from FunctionContext
Date Wed, 25 Oct 2017 20:44:00 GMT


ASF subversion and git services commented on GEODE-3299:

Commit 0ccf9fab8c7ccf109b91f17272c9e29f0854545e in geode's branch refs/heads/develop from
[;h=0ccf9fa ]

GEODE-3299: throw CacheClosedException in FunctionContext.getCache() (#975)

* GEODE-3299: throw CacheClosedException when FunctionContext.getCache() == null

Gfsh functions handle CacheClosedException, whereas
NullPointerExceptions that resulted from returning null are not.

* GEODE-3299: Updates to ShowMissingDiskStoresFunction

Adjusted test in ShowMissingDiskStoresDUniotTest to expect the exception instead of empty
results. Also removed redundant fail() assertions from the tests.

> Gfsh functions should acquire Cache from FunctionContext
> --------------------------------------------------------
>                 Key: GEODE-3299
>                 URL:
>             Project: Geode
>          Issue Type: Bug
>          Components: gfsh
>            Reporter: Kirk Lund
>            Assignee: Kenneth Howe
>             Fix For: 1.4.0
> Gfsh functions in geode-core 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

View raw message