You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2018/04/24 16:35:59 UTC

[geode] branch develop updated: GEODE-5020: calculateThreadIdExpiryTime during ThreadIdentifierCustomExpiry ctor

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

klund 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 eb0bb89  GEODE-5020: calculateThreadIdExpiryTime during ThreadIdentifierCustomExpiry ctor
eb0bb89 is described below

commit eb0bb8968c50a4580e920e214c8c96b92c9b6171
Author: Kirk Lund <kl...@apache.org>
AuthorDate: Mon Apr 23 09:47:17 2018 -0700

    GEODE-5020: calculateThreadIdExpiryTime during ThreadIdentifierCustomExpiry ctor
---
 .../java/org/apache/geode/internal/cache/ha/HARegionQueue.java    | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/ha/HARegionQueue.java b/geode-core/src/main/java/org/apache/geode/internal/cache/ha/HARegionQueue.java
index 8a5ccbb..6d595c2 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/ha/HARegionQueue.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/ha/HARegionQueue.java
@@ -3799,12 +3799,18 @@ public class HARegionQueue implements RegionQueue {
 
     private static volatile ExpirationAttributes testExpAtts;
 
+    private final int expTime;
+
+    ThreadIdentifierCustomExpiry() {
+      expTime = calculateThreadIdExpiryTime();
+    }
+
     @Override
     public ExpirationAttributes getExpiry(Region.Entry entry) {
       // Use key to determine expiration.
       Object key = entry.getKey();
       if (key instanceof ThreadIdentifier) {
-        final int expTime = calculateThreadIdExpiryTime();
+        // TODO: inject subclass of ThreadIdentifierCustomExpiry and move next block to test class
         if (expTime != DEFAULT_THREAD_ID_EXPIRY_TIME) {
           // This should only happen in unit test code
           ExpirationAttributes result = testExpAtts;

-- 
To stop receiving notification emails like this one, please contact
klund@apache.org.