You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ds...@apache.org on 2016/03/08 01:25:09 UTC

[02/11] incubator-geode git commit: GEODE-1029 fixed test issue, where system is not working properly.

GEODE-1029 fixed test issue, where system is not working properly.


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/e1a0830e
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/e1a0830e
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/e1a0830e

Branch: refs/heads/feature/GEODE-982
Commit: e1a0830e2eda15cf7a8ec2281b521619333c8e53
Parents: 3996940
Author: Hitesh Khamesra <hk...@pivotal.io>
Authored: Tue Mar 1 14:26:48 2016 -0800
Committer: Hitesh Khamesra <hk...@pivotal.io>
Committed: Wed Mar 2 09:36:34 2016 -0800

----------------------------------------------------------------------
 .../gemfire/internal/cache/CachedDeserializableFactory.java     | 4 ++--
 .../gemfire/cache30/DistributedMulticastRegionDUnitTest.java    | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/e1a0830e/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/CachedDeserializableFactory.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/CachedDeserializableFactory.java b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/CachedDeserializableFactory.java
index 9120e37..e1bf984 100644
--- a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/CachedDeserializableFactory.java
+++ b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/CachedDeserializableFactory.java
@@ -35,8 +35,8 @@ import com.gemstone.gemfire.pdx.PdxInstance;
  *
  */
 public class CachedDeserializableFactory {
-  private static final boolean PREFER_DESERIALIZED = ! Boolean.getBoolean("gemfire.PREFER_SERIALIZED");
-  private static final boolean STORE_ALL_VALUE_FORMS = Boolean.getBoolean("gemfire.STORE_ALL_VALUE_FORMS");
+  public static boolean PREFER_DESERIALIZED = ! Boolean.getBoolean("gemfire.PREFER_SERIALIZED");
+  public static boolean STORE_ALL_VALUE_FORMS = Boolean.getBoolean("gemfire.STORE_ALL_VALUE_FORMS");
 
   /**
    * Currently GFE always wants a CachedDeserializable wrapper.

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/e1a0830e/geode-core/src/test/java/com/gemstone/gemfire/cache30/DistributedMulticastRegionDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/com/gemstone/gemfire/cache30/DistributedMulticastRegionDUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/cache30/DistributedMulticastRegionDUnitTest.java
index 72b7941..9a0a6dd 100755
--- a/geode-core/src/test/java/com/gemstone/gemfire/cache30/DistributedMulticastRegionDUnitTest.java
+++ b/geode-core/src/test/java/com/gemstone/gemfire/cache30/DistributedMulticastRegionDUnitTest.java
@@ -34,6 +34,7 @@ import com.gemstone.gemfire.distributed.internal.InternalLocator;
 import com.gemstone.gemfire.distributed.internal.ReplyException;
 import com.gemstone.gemfire.distributed.internal.SharedConfiguration;
 import com.gemstone.gemfire.internal.AvailablePortHelper;
+import com.gemstone.gemfire.internal.cache.CachedDeserializableFactory;
 import com.gemstone.gemfire.internal.cache.GemFireCacheImpl;
 import com.gemstone.gemfire.internal.cache.OffHeapTestUtil;
 import com.gemstone.gemfire.pdx.PdxReader;
@@ -159,7 +160,7 @@ public class DistributedMulticastRegionDUnitTest extends CacheTestCase {
       
       SerializableRunnable setSysProp = new CacheSerializableRunnable("Create Region") {
         public void run2() throws CacheException {
-          System.setProperty("gemfire.STORE_ALL_VALUE_FORMS", "true");
+          CachedDeserializableFactory.STORE_ALL_VALUE_FORMS = true;
         }
       };
       
@@ -225,7 +226,7 @@ public class DistributedMulticastRegionDUnitTest extends CacheTestCase {
     }finally {
       SerializableRunnable unsetSysProp = new CacheSerializableRunnable("Create Region") {
         public void run2() throws CacheException {
-          System.setProperty("gemfire.STORE_ALL_VALUE_FORMS", "false");          
+          CachedDeserializableFactory.STORE_ALL_VALUE_FORMS = false;
         }
       };
       vm0.invoke(unsetSysProp);