You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by zh...@apache.org on 2018/10/10 23:36:28 UTC

[geode] branch develop updated: GEODE-5252: handleCacheRemoval() could wait for distribution's replies. (#2590)

This is an automated email from the ASF dual-hosted git repository.

zhouxj pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new cc6f0de  GEODE-5252: handleCacheRemoval() could wait for distribution's replies. (#2590)
cc6f0de is described below

commit cc6f0de3796241d940064f405fca3eb6711f4784
Author: Xiaojian Zhou <ge...@users.noreply.github.com>
AuthorDate: Wed Oct 10 16:36:20 2018 -0700

    GEODE-5252: handleCacheRemoval() could wait for distribution's replies. (#2590)
    
    Should not block the system notifications for membership changes.
    
    Co-authored-by: Sai Boorlagadda <sb...@pivotal.io>
---
 .../org/apache/geode/management/internal/beans/ManagementListener.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/beans/ManagementListener.java b/geode-core/src/main/java/org/apache/geode/management/internal/beans/ManagementListener.java
index 41bd7c4..e5b6ec7 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/beans/ManagementListener.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/beans/ManagementListener.java
@@ -109,7 +109,7 @@ public class ManagementListener implements ResourceEventsListener {
     try {
       if (event == ResourceEvent.CACHE_CREATE || event == ResourceEvent.CACHE_REMOVE) {
         readWriteLock.writeLock().lock();
-      } else {
+      } else if (event != ResourceEvent.SYSTEM_ALERT) {
         readWriteLock.readLock().lock();
       }
       switch (event) {