[jira] [Commented] (GEODE-1752) memory leak in GMSMembershipManager

["Kamilla Aslami (Jira)" <ji...@apache.org>]

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

Kamilla Aslami commented on GEODE-1752:
---------------------------------------

Latches should be removed in GMSMembership.processView():
{code:java}
if (isSecure) {
  CountDownLatch currentLatch;
  if ((currentLatch = memberLatch.get(m)) != null) {
    currentLatch.countDown();
  }
}
{code}

> memory leak in GMSMembershipManager
> -----------------------------------
>
>                 Key: GEODE-1752
>                 URL: https://issues.apache.org/jira/browse/GEODE-1752
>             Project: Geode
>          Issue Type: Bug
>          Components: membership
>            Reporter: Bruce J Schuchardt
>            Priority: Major
>
> GMSMembershipManager has a TODO to rewrite waitForMembershipCheck to wait for a particular membership view to be installed.  It currently uses a map of memberID->CountDownLatch to wait for a member but the latches are never removed from the map.
> DistributionManager already has a waitForViewInstallation method that does something similar and it could be replaced by a call to the new GMSMembershipManager method.



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