[ 
https://issues.apache.org/jira/browse/GEODE-7221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16934606#comment-16934606
 ] 

ASF subversion and git services commented on GEODE-7221:
--------------------------------------------------------

Commit 6d3dd7b37cf4b7f64b383b9e8087fbf21c0303f6 in geode's branch 
refs/heads/develop from Kirk Lund
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=6d3dd7b ]

GEODE-7221: Set hasOwnStats true for management regions

Ensure that managements regions close CachePerfStats by setting
hasOwnStats to true from FederatingManager and LocalManager.


> Management regions never close dedicated CachePerfStats
> -------------------------------------------------------
>
>                 Key: GEODE-7221
>                 URL: https://issues.apache.org/jira/browse/GEODE-7221
>             Project: Geode
>          Issue Type: Bug
>          Components: management
>    Affects Versions: 1.0.0-incubating, 1.1.0, 1.1.1, 1.2.0, 1.3.0, 1.2.1, 
> 1.4.0, 1.5.0, 1.6.0, 1.7.0, 1.8.0, 1.9.0, 1.9.1, 1.10.0
>            Reporter: Kirk Lund
>            Assignee: Kirk Lund
>            Priority: Minor
>             Fix For: 1.11.0
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> The FederatingManager and LocalManager classes create the monitoring and 
> notification regions for management of each member. They are constructing 
> injecting HasCachePerfStats instances with their own dedicated instances of 
> CachePerfStats into InternalRegionArgs:
> {noformat}
> InternalRegionArgs.setCachePerfStatsHolder(HasCachePerfStats)
> {noformat}
> Over in LocalRegion, the field hasOwnStats is set to false:
> {noformat}
>     if (internalRegionArgs.getCachePerfStatsHolder() != null) {
>       hasOwnStats = false;
>       cachePerfStats = 
> internalRegionArgs.getCachePerfStatsHolder().getCachePerfStats();
> {noformat}
> Which results in region destroy not closing the CachePerfStats.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to