You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by sa...@apache.org on 2021/04/13 14:28:21 UTC

[geode] 11/11: GEODE-8781: Added ReconnectListener to PeerToPeerSessionCache

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

sabbey37 pushed a commit to branch support/1.13
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 31d2613189f623373fb997e88af9fca7746c2a0f
Author: Barry Oglesby <bo...@users.noreply.github.com>
AuthorDate: Mon Dec 14 12:07:45 2020 -0800

    GEODE-8781: Added ReconnectListener to PeerToPeerSessionCache
    
    (cherry picked from commit bda0d3fbf13ebdf37db8d710cf7f4d4b892a1646)
---
 .../modules/session/catalina/DeltaSessionManager.java |  6 +++++-
 .../session/catalina/PeerToPeerSessionCache.java      | 19 +++++++++++++++++++
 2 files changed, 24 insertions(+), 1 deletion(-)

diff --git a/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/DeltaSessionManager.java b/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/DeltaSessionManager.java
index a42b0d8..997c424 100644
--- a/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/DeltaSessionManager.java
+++ b/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/DeltaSessionManager.java
@@ -551,7 +551,11 @@ public abstract class DeltaSessionManager<CommitSessionValveT extends AbstractCo
     TimerTask task = new TimerTask() {
       @Override
       public void run() {
-        int currentActiveSessions = getSessionCache().size();
+        int currentActiveSessions = 0;
+        try {
+          currentActiveSessions = getSessionCache().size();
+        } catch (Exception ignore) {
+        }
         if (currentActiveSessions > getMaxActive()) {
           setMaxActive(currentActiveSessions);
           if (getLogger().isDebugEnabled()) {
diff --git a/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/PeerToPeerSessionCache.java b/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/PeerToPeerSessionCache.java
index d5031c1..04e197d 100644
--- a/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/PeerToPeerSessionCache.java
+++ b/extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/PeerToPeerSessionCache.java
@@ -27,6 +27,8 @@ import org.apache.geode.cache.execute.Execution;
 import org.apache.geode.cache.execute.Function;
 import org.apache.geode.cache.execute.FunctionService;
 import org.apache.geode.cache.execute.ResultCollector;
+import org.apache.geode.distributed.internal.InternalDistributedSystem;
+import org.apache.geode.internal.cache.InternalCache;
 import org.apache.geode.modules.session.catalina.callback.LocalSessionCacheLoader;
 import org.apache.geode.modules.session.catalina.callback.LocalSessionCacheWriter;
 import org.apache.geode.modules.session.catalina.callback.SessionExpirationCacheListener;
@@ -47,6 +49,7 @@ public class PeerToPeerSessionCache extends AbstractSessionCache {
   public PeerToPeerSessionCache(SessionManager sessionManager, Cache cache) {
     super(sessionManager);
     this.cache = cache;
+    addReconnectListener();
   }
 
   @Override
@@ -242,4 +245,20 @@ public class PeerToPeerSessionCache extends AbstractSessionCache {
   Execution getExecutionForFunctionOnMembersWithArguments(Object[] arguments) {
     return FunctionService.onMembers().setArguments(arguments);
   }
+
+  private void addReconnectListener() {
+    InternalDistributedSystem.addReconnectListener(
+        new InternalDistributedSystem.ReconnectListener() {
+          @Override
+          public void onReconnect(InternalDistributedSystem oldSystem,
+              InternalDistributedSystem newSystem) {
+            reinitialize(newSystem.getCache());
+          }
+        });
+  }
+
+  private void reinitialize(InternalCache reconnectedCache) {
+    this.cache = reconnectedCache;
+    initialize();
+  }
 }