From 7a448183c053b499529e881c147b65a1c0cdaeb5 Mon Sep 17 00:00:00 2001 From: John Blum Date: Fri, 14 Dec 2018 02:55:41 -0800 Subject: [PATCH] Change the default value for register interest receive values to true, matching the GemFire/Geode default. Resolves gh-21. --- .../gemfire/AbstractGemFireOperationsSessionRepository.java | 2 +- .../AbstractGemFireOperationsSessionRepositoryTests.java | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/spring-session-data-geode/src/main/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepository.java b/spring-session-data-geode/src/main/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepository.java index aac756c..d45cc55 100644 --- a/spring-session-data-geode/src/main/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepository.java +++ b/spring-session-data-geode/src/main/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepository.java @@ -107,7 +107,7 @@ public abstract class AbstractGemFireOperationsSessionRepository private static final boolean DEFAULT_REGISTER_INTEREST_ENABLED = false; private static final boolean DEFAULT_REGISTER_INTEREST_DURABILITY = false; - private static final boolean DEFAULT_REGISTER_INTEREST_RECEIVE_VALUES = false; + private static final boolean DEFAULT_REGISTER_INTEREST_RECEIVE_VALUES = true; private static final AtomicBoolean usingDataSerialization = new AtomicBoolean(false); diff --git a/spring-session-data-geode/src/test/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepositoryTests.java b/spring-session-data-geode/src/test/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepositoryTests.java index c881e0e..5e07da1 100644 --- a/spring-session-data-geode/src/test/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepositoryTests.java +++ b/spring-session-data-geode/src/test/java/org/springframework/session/data/gemfire/AbstractGemFireOperationsSessionRepositoryTests.java @@ -635,7 +635,7 @@ public class AbstractGemFireOperationsSessionRepositoryTests { verify(this.sessionRepository, times(1)).registerInterest(eq("1")); verify(this.sessionRepository, times(1)).isRegisterInterestEnabled(); verify(this.mockRegion, times(1)) - .registerInterest(eq("1"), eq(InterestResultPolicy.NONE), eq(false), eq(false)); + .registerInterest(eq("1"), eq(InterestResultPolicy.NONE), eq(false), eq(true)); } @Test @@ -676,7 +676,7 @@ public class AbstractGemFireOperationsSessionRepositoryTests { verify(this.sessionRepository, times(2)).registerInterest(eq("1")); verify(this.sessionRepository, times(2)).isRegisterInterestEnabled(); verify(this.mockRegion, times(1)) - .registerInterest(eq("1"), eq(InterestResultPolicy.NONE), eq(false), eq(false)); + .registerInterest(eq("1"), eq(InterestResultPolicy.NONE), eq(false), eq(true)); } @Test