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 2017/05/15 22:09:11 UTC

[4/5] geode git commit: GEODE-2929: remove final from internal classes and tests

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AppCacheSnapshotMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AppCacheSnapshotMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AppCacheSnapshotMessage.java
index 7e08aa7..72f6903 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AppCacheSnapshotMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AppCacheSnapshotMessage.java
@@ -24,7 +24,7 @@ import org.apache.geode.cache.*;
 import org.apache.geode.*;
 // import java.util.*;
 
-public final class AppCacheSnapshotMessage extends RegionAdminMessage {
+public class AppCacheSnapshotMessage extends RegionAdminMessage {
   // private int numResults;
   // private static Map consolesToSnapshots = new HashMap();
   private int snapshotId;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerRequest.java
index ae1d171..f65b09d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  *
  * @since GemFire 4.0
  */
-public final class BridgeServerRequest extends AdminRequest {
+public class BridgeServerRequest extends AdminRequest {
 
   /** Add a new bridge server */
   static final int ADD_OPERATION = 10;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerResponse.java
index 4e915d6..ddc102d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/BridgeServerResponse.java
@@ -33,7 +33,7 @@ import org.apache.geode.internal.cache.InternalCache;
  *
  * @since GemFire 4.0
  */
-public final class BridgeServerResponse extends AdminResponse {
+public class BridgeServerResponse extends AdminResponse {
 
   /** Information about the bridge server that was operated on */
   private RemoteBridgeServer bridgeInfo;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigRequest.java
index b2c5c42..b541658 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigRequest.java
@@ -31,7 +31,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class CacheConfigRequest extends AdminRequest {
+public class CacheConfigRequest extends AdminRequest {
   private byte attributeCode;
   private int newValue;
   private int cacheId;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigResponse.java
index aedf67e..2e9c37c 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheConfigResponse.java
@@ -31,7 +31,7 @@ import org.apache.geode.internal.cache.InternalCache;
  * 
  * @since GemFire 3.5
  */
-public final class CacheConfigResponse extends AdminResponse {
+public class CacheConfigResponse extends AdminResponse {
 
   private RemoteCacheInfo info;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheDisplay.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheDisplay.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheDisplay.java
index fe596db..c842a19 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheDisplay.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheDisplay.java
@@ -20,7 +20,7 @@ package org.apache.geode.internal.admin.remote;
 import org.apache.geode.internal.admin.GemFireVM;
 import org.apache.geode.internal.i18n.LocalizedStrings;
 
-public final class CacheDisplay {
+public class CacheDisplay {
   public static Object getCachedObjectDisplay(Object obj, int inspectionType) {
     switch (inspectionType) {
       case GemFireVM.LIGHTWEIGHT_CACHE_VALUE:

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoRequest.java
index 9a2c2e8..21ad6c2 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoRequest.java
@@ -29,7 +29,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class CacheInfoRequest extends AdminRequest {
+public class CacheInfoRequest extends AdminRequest {
   /**
    * Returns a <code>CacheInfoRequest</code>.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoResponse.java
index 5a5c36c..578252e 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CacheInfoResponse.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.cache.InternalCache;
  * 
  * @since GemFire 3.5
  */
-public final class CacheInfoResponse extends AdminResponse {
+public class CacheInfoResponse extends AdminResponse {
 
   private RemoteCacheInfo info;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerRequest.java
index 2559ef2..d96d567 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  * A message that is sent to a particular distribution manager to get rid of a previously added
  * statistic listener.
  */
-public final class CancelStatListenerRequest extends AdminRequest {
+public class CancelStatListenerRequest extends AdminRequest {
   // instance variables
   private int listenerId;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerResponse.java
index 394e5b6..903f6fb 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancelStatListenerResponse.java
@@ -29,7 +29,7 @@ import org.apache.geode.internal.statistics.GemFireStatSampler;
  * A message that is sent to a particular distribution manager to get its current
  * <code>RemoteCancelStatListener</code>.
  */
-public final class CancelStatListenerResponse extends AdminResponse {
+public class CancelStatListenerResponse extends AdminResponse {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancellationMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancellationMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancellationMessage.java
index 462234b..5bab9f1 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancellationMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/CancellationMessage.java
@@ -27,7 +27,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
 /**
  * A message that is sent to a particular distribution manager to cancel an admin request
  */
-public final class CancellationMessage extends PooledDistributionMessage {
+public class CancellationMessage extends PooledDistributionMessage {
   // instance variables
   private int msgToCancel;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyEntryMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyEntryMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyEntryMessage.java
index 9dda2c9..ceb940f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyEntryMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyEntryMessage.java
@@ -36,7 +36,7 @@ import org.apache.geode.internal.logging.log4j.LocalizedMessage;
  * A message that is sent to a particular distribution manager to let it know that the sender is an
  * administation console that just connected.
  */
-public final class DestroyEntryMessage extends RegionAdminMessage {
+public class DestroyEntryMessage extends RegionAdminMessage {
 
   private static final Logger logger = LogService.getLogger();
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyRegionMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyRegionMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyRegionMessage.java
index 3e401ff..e6ed87c 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyRegionMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/DestroyRegionMessage.java
@@ -36,7 +36,7 @@ import org.apache.geode.internal.logging.log4j.LocalizedMessage;
  * A message that is sent to a particular distribution manager to let it know that the sender is an
  * administation console that just connected.
  */
-public final class DestroyRegionMessage extends RegionAdminMessage {
+public class DestroyRegionMessage extends RegionAdminMessage {
 
   private static final Logger logger = LogService.getLogger();
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoRequest.java
index 6272454..22e8378 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoRequest.java
@@ -23,7 +23,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
 import java.io.*;
 // import java.util.*;
 
-public final class FetchDistLockInfoRequest extends AdminRequest {
+public class FetchDistLockInfoRequest extends AdminRequest {
   /**
    * Returns a <code>FetchDistLockInfoRequest</code> to be sent to the specified recipient.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoResponse.java
index 6de15dd..ba6a0c4 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchDistLockInfoResponse.java
@@ -26,7 +26,7 @@ import java.io.*;
 import java.util.*;
 import org.apache.geode.distributed.internal.membership.*;
 
-public final class FetchDistLockInfoResponse extends AdminResponse {
+public class FetchDistLockInfoResponse extends AdminResponse {
   // instance variables
   DLockInfo[] lockInfos;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisRequest.java
index 09369ba..b491632 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class FetchHealthDiagnosisRequest extends AdminRequest {
+public class FetchHealthDiagnosisRequest extends AdminRequest {
   // instance variables
   private int id = 0;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisResponse.java
index 0f44767..55bdc64 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHealthDiagnosisResponse.java
@@ -30,7 +30,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * 
  * @since GemFire 3.5
  */
-public final class FetchHealthDiagnosisResponse extends AdminResponse {
+public class FetchHealthDiagnosisResponse extends AdminResponse {
   // instance variables
   String[] diagnosis;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostRequest.java
index 4868158..74e5fec 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostRequest.java
@@ -24,7 +24,7 @@ import java.io.*;
 /**
  * A message that is sent to a particular distribution manager to get its current information
  */
-public final class FetchHostRequest extends AdminRequest {
+public class FetchHostRequest extends AdminRequest {
   /**
    * Returns a <code>FetchHostRequest</code> to be sent to the specified recipient.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostResponse.java
index 9a771f8..9ca66ca 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchHostResponse.java
@@ -41,7 +41,7 @@ import java.net.UnknownHostException;
  * A message that is sent to a particular distribution manager to get its current
  * {@link org.apache.geode.internal.Config}
  */
-public final class FetchHostResponse extends AdminResponse {
+public class FetchHostResponse extends AdminResponse {
   private static final Logger logger = LogService.getLogger();
 
   // instance variables

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesRequest.java
index efddcd2..e4cf695 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesRequest.java
@@ -23,7 +23,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
 import java.io.*;
 // import java.util.*;
 
-public final class FetchResourceAttributesRequest extends AdminRequest {
+public class FetchResourceAttributesRequest extends AdminRequest {
 
   // instance variables
   private long resourceUniqueId;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesResponse.java
index f6c7c8d..7d0fbb8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchResourceAttributesResponse.java
@@ -21,7 +21,7 @@ import org.apache.geode.*;
 import java.io.*;
 import org.apache.geode.distributed.internal.membership.*;
 
-public final class FetchResourceAttributesResponse extends AdminResponse {
+public class FetchResourceAttributesResponse extends AdminResponse {
 
   // instance variables
   private RemoteStat[] stats;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsRequest.java
index 94677b8..36994ad 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsRequest.java
@@ -23,7 +23,7 @@ import org.apache.geode.distributed.internal.*;
 import java.io.*;
 // import java.util.*;
 
-public final class FetchStatsRequest extends AdminRequest {
+public class FetchStatsRequest extends AdminRequest {
 
   private String statisticsTypeName;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsResponse.java
index 08c96d6..ab122c6 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchStatsResponse.java
@@ -32,7 +32,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * Provides a response of remote statistic resources for a <code>FetchStatsRequest</code>
  *
  */
-public final class FetchStatsResponse extends AdminResponse {
+public class FetchStatsResponse extends AdminResponse {
 
   // instance variables
   private RemoteStatResource[] stats;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgRequest.java
index 9b8a98f..a06356a 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgRequest.java
@@ -25,7 +25,7 @@ import java.io.*;
  * A message that is sent to a particular distribution manager to get its current
  * {@link org.apache.geode.internal.Config}.
  */
-public final class FetchSysCfgRequest extends AdminRequest {
+public class FetchSysCfgRequest extends AdminRequest {
   /**
    * Returns a <code>FetchSysCfgRequest</code> to be sent to the specified recipient.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgResponse.java
index f9f9449..b50a3b5 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FetchSysCfgResponse.java
@@ -26,7 +26,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * A message that is sent to a particular distribution manager to get its current {@link Config}.
  */
-public final class FetchSysCfgResponse extends AdminResponse {
+public class FetchSysCfgResponse extends AdminResponse {
   // instance variables
   Config sc;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FlushAppCacheSnapshotMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FlushAppCacheSnapshotMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FlushAppCacheSnapshotMessage.java
index a5bfb64..fddafe5 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FlushAppCacheSnapshotMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/FlushAppCacheSnapshotMessage.java
@@ -26,7 +26,7 @@ import java.io.*;
  * A message to cause a remote application to release any snapshot info it was holding on behalf of
  * a console.
  */
-public final class FlushAppCacheSnapshotMessage extends PooledDistributionMessage {
+public class FlushAppCacheSnapshotMessage extends PooledDistributionMessage {
 
   public static FlushAppCacheSnapshotMessage create() {
     FlushAppCacheSnapshotMessage m = new FlushAppCacheSnapshotMessage();

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/HealthListenerMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/HealthListenerMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/HealthListenerMessage.java
index 653cf6a..b3b309a 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/HealthListenerMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/HealthListenerMessage.java
@@ -31,8 +31,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class HealthListenerMessage extends PooledDistributionMessage
-    implements AdminMessageType {
+public class HealthListenerMessage extends PooledDistributionMessage implements AdminMessageType {
   // instance variables
   private int listenerId;
   private GemFireHealth.Health status;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoRequest.java
index abcabb3..1ba0088 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoRequest.java
@@ -26,7 +26,7 @@ import java.io.*;
 /**
  * A message that is sent to a particular distribution manager to get its current license info.
  */
-public final class LicenseInfoRequest extends AdminRequest {
+public class LicenseInfoRequest extends AdminRequest {
   /**
    * Returns a <code>LicenseInfoRequest</code>.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoResponse.java
index 73c847a..41bfa9c 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/LicenseInfoResponse.java
@@ -30,7 +30,7 @@ import java.util.Properties;
 /**
  * A message that is sent in response to a {@link LicenseInfoRequest}.
  */
-public final class LicenseInfoResponse extends AdminResponse {
+public class LicenseInfoResponse extends AdminResponse {
   private static final Logger logger = LogService.getLogger();
 
   // instance variables

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsRequest.java
index 11c5336..86167a8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  * A message that is sent to a particular app vm to request the value, stats, and attributes of a
  * given region entry.
  */
-public final class ObjectDetailsRequest extends RegionAdminRequest implements Cancellable {
+public class ObjectDetailsRequest extends RegionAdminRequest implements Cancellable {
   // instance variables
   private Object objName;
   private int inspectionType;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsResponse.java
index 6c54a42..af056e4 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectDetailsResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Responds to {@link ObjectDetailsRequest}.
  */
-public final class ObjectDetailsResponse extends AdminResponse implements Cancellable {
+public class ObjectDetailsResponse extends AdminResponse implements Cancellable {
   // instance variables
   private Object objectValue;
   private Object userAttribute;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesRequest.java
index aaabec6..6424e19 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  * A message that is sent to a particular app vm to request all the subregions of a given parent
  * region.
  */
-public final class ObjectNamesRequest extends RegionAdminRequest implements Cancellable {
+public class ObjectNamesRequest extends RegionAdminRequest implements Cancellable {
   // instance variables
   private transient boolean cancelled;
   private transient ObjectNamesResponse resp;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesResponse.java
index 5a0d076..e6832c5 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ObjectNamesResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Responds to {@link ObjectNamesResponse}.
  */
-public final class ObjectNamesResponse extends AdminResponse implements Cancellable {
+public class ObjectNamesResponse extends AdminResponse implements Cancellable {
   // instance variables
   private HashSet objectNames;
   private transient boolean cancelled;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesRequest.java
index ce0af24..4592161 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesRequest.java
@@ -26,7 +26,7 @@ import java.io.*;
 /**
  * A message that is sent to a particular app vm to request the region statistics of a given region.
  */
-public final class RegionAttributesRequest extends RegionAdminRequest {
+public class RegionAttributesRequest extends RegionAdminRequest {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesResponse.java
index 99d0ee1..4ef4cd2 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionAttributesResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Responds to {@link RegionAttributesResponse}.
  */
-public final class RegionAttributesResponse extends AdminResponse {
+public class RegionAttributesResponse extends AdminResponse {
   // instance variables
   private RemoteRegionAttributes attributes;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionRequest.java
index 5850767..c63a8ce 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionRequest.java
@@ -28,7 +28,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class RegionRequest extends AdminRequest {
+public class RegionRequest extends AdminRequest {
 
   /** Request to get a region */
   static final int GET_REGION = 10;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionResponse.java
index a7971a7..b3b678e 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionResponse.java
@@ -31,7 +31,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * 
  * @since GemFire 3.5
  */
-public final class RegionResponse extends AdminResponse {
+public class RegionResponse extends AdminResponse {
   // instance variables
 
   /** The name of the region returned in the response. */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeRequest.java
index 96b177b..453e076 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeRequest.java
@@ -28,7 +28,7 @@ import java.io.*;
  * A message that is sent to a particular app vm to request all the subregions of a given parent
  * region.
  */
-public final class RegionSizeRequest extends RegionAdminRequest implements Cancellable {
+public class RegionSizeRequest extends RegionAdminRequest implements Cancellable {
   private transient boolean cancelled;
   private transient RegionSizeResponse resp;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeResponse.java
index c6c1ab3..d0c28b3 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionSizeResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Responds to {@link RegionSizeRequest}.
  */
-public final class RegionSizeResponse extends AdminResponse implements Cancellable {
+public class RegionSizeResponse extends AdminResponse implements Cancellable {
   // instance variables
   private int entryCount;
   private int subregionCount;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsRequest.java
index c4d1791..c95f9db 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsRequest.java
@@ -26,7 +26,7 @@ import java.io.*;
 /**
  * A message that is sent to a particular app vm to request the region statistics of a given region.
  */
-public final class RegionStatisticsRequest extends RegionAdminRequest {
+public class RegionStatisticsRequest extends RegionAdminRequest {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsResponse.java
index 8480842..e2c919d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RegionStatisticsResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Responds to {@link RegionStatisticsResponse}.
  */
-public final class RegionStatisticsResponse extends AdminResponse {
+public class RegionStatisticsResponse extends AdminResponse {
   // instance variables
   RemoteCacheStatistics regionStatistics;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteApplicationVM.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteApplicationVM.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteApplicationVM.java
index 1679705..cfbdd5b 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteApplicationVM.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteApplicationVM.java
@@ -22,7 +22,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Represents an application VM (member of the distributed system).
  */
-public final class RemoteApplicationVM extends RemoteGemFireVM implements ApplicationVM {
+public class RemoteApplicationVM extends RemoteGemFireVM implements ApplicationVM {
 
   // constructors
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteGfManagerAgent.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteGfManagerAgent.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteGfManagerAgent.java
index e23f2d2..64b5925 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteGfManagerAgent.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteGfManagerAgent.java
@@ -49,7 +49,7 @@ import java.util.concurrent.*;
 public
 // Note that since we export the instances in a public list,
 // I'm not permitting subclasses
-final class RemoteGfManagerAgent implements GfManagerAgent {
+class RemoteGfManagerAgent implements GfManagerAgent {
 
   private static final Logger logger = LogService.getLogger();
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerRequest.java
index 08d94e6..570b6fa 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerRequest.java
@@ -29,7 +29,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class RemoveHealthListenerRequest extends AdminRequest {
+public class RemoveHealthListenerRequest extends AdminRequest {
   // instance variables
   private int id;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerResponse.java
index f8a7b80..894a2a6 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoveHealthListenerResponse.java
@@ -29,7 +29,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * 
  * @since GemFire 3.5
  */
-public final class RemoveHealthListenerResponse extends AdminResponse {
+public class RemoveHealthListenerResponse extends AdminResponse {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusRequest.java
index 6752e97..e381f6e 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusRequest.java
@@ -29,7 +29,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class ResetHealthStatusRequest extends AdminRequest {
+public class ResetHealthStatusRequest extends AdminRequest {
   // instance variables
   private int id;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusResponse.java
index b18ac07..66ca180 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/ResetHealthStatusResponse.java
@@ -29,7 +29,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * 
  * @since GemFire 3.5
  */
-public final class ResetHealthStatusResponse extends AdminResponse {
+public class ResetHealthStatusResponse extends AdminResponse {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionRequest.java
index c927e13..2651f53 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  * A message that is sent to a particular application to get its root region(s). All it really needs
  * to find out is if the app has created a cache.
  */
-public final class RootRegionRequest extends AdminRequest {
+public class RootRegionRequest extends AdminRequest {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionResponse.java
index 78ac174..3dad8f7 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RootRegionResponse.java
@@ -34,7 +34,7 @@ import org.apache.geode.internal.cache.InternalCache;
 /**
  * Responds to {@link RootRegionResponse}.
  */
-public final class RootRegionResponse extends AdminResponse {
+public class RootRegionResponse extends AdminResponse {
 
   private String[] regions;
   private String[] userAttrs;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SnapshotResultMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SnapshotResultMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SnapshotResultMessage.java
index de2fba9..9b2aa02 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SnapshotResultMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SnapshotResultMessage.java
@@ -24,8 +24,7 @@ import org.apache.geode.cache.*;
 import java.io.*;
 // import java.util.*;
 
-public final class SnapshotResultMessage extends PooledDistributionMessage
-    implements AdminMessageType {
+public class SnapshotResultMessage extends PooledDistributionMessage implements AdminMessageType {
   private CacheSnapshot results;
   private int snapshotId;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StatListenerMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StatListenerMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StatListenerMessage.java
index 28ab9cf..e4142b9 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StatListenerMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StatListenerMessage.java
@@ -27,8 +27,7 @@ import java.io.*;
  * A message that is sent to a particular console distribution manager when changes have been
  * detected that will be of interest to registered stat listeners.
  */
-public final class StatListenerMessage extends PooledDistributionMessage
-    implements AdminMessageType {
+public class StatListenerMessage extends PooledDistributionMessage implements AdminMessageType {
 
   // instance variables
   private long timestamp;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgRequest.java
index 5049882..340794f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgRequest.java
@@ -28,7 +28,7 @@ import java.io.*;
  * A message that is sent to a particular distribution manager to modify its current
  * {@link org.apache.geode.internal.Config}.
  */
-public final class StoreSysCfgRequest extends AdminRequest {
+public class StoreSysCfgRequest extends AdminRequest {
   // instance variables
   Config sc;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgResponse.java
index b4d6dfe..c1d0048 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/StoreSysCfgResponse.java
@@ -26,7 +26,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * A message that is sent to a particular distribution manager to modify its current
  * {@link org.apache.geode.internal.Config}.
  */
-public final class StoreSysCfgResponse extends AdminResponse {
+public class StoreSysCfgResponse extends AdminResponse {
   // instance variables
 
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionRequest.java
index 0fc05fc..e9216b5 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionRequest.java
@@ -27,7 +27,7 @@ import java.io.*;
  * A message that is sent to a particular app vm to request all the subregions of a given parent
  * region.
  */
-public final class SubRegionRequest extends RegionAdminRequest {
+public class SubRegionRequest extends RegionAdminRequest {
   // instance variables
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionResponse.java
index 01b98df..267648f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/SubRegionResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * Responds to {@link SubRegionResponse}.
  */
-public final class SubRegionResponse extends AdminResponse {
+public class SubRegionResponse extends AdminResponse {
   // instance variables
   String[] subRegionNames;
   String[] userAttributes;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogRequest.java
index 6665ad9..8ca369d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogRequest.java
@@ -23,7 +23,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
 import java.io.*;
 // import java.util.*;
 
-public final class TailLogRequest extends AdminRequest {
+public class TailLogRequest extends AdminRequest {
   public static TailLogRequest create() {
     TailLogRequest m = new TailLogRequest();
     return m;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogResponse.java
index c5eff6f..5d94bc7 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/TailLogResponse.java
@@ -33,7 +33,7 @@ import java.io.*;
 // import org.apache.geode.distributed.DistributedSystem;
 // import java.util.*;
 
-public final class TailLogResponse extends AdminResponse {
+public class TailLogResponse extends AdminResponse {
   private static final Logger logger = LogService.getLogger();
 
   private String tail;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoRequest.java
index 87f3824..1079025 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoRequest.java
@@ -28,7 +28,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class VersionInfoRequest extends AdminRequest {
+public class VersionInfoRequest extends AdminRequest {
   /**
    * Returns a <code>VersionInfoRequest</code>.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoResponse.java
index 1887fce..a472af8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/VersionInfoResponse.java
@@ -29,7 +29,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * 
  * @since GemFire 3.5
  */
-public final class VersionInfoResponse extends AdminResponse {
+public class VersionInfoResponse extends AdminResponse {
   // instance variables
   private String verInfo;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionDecoratorImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionDecoratorImpl.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionDecoratorImpl.java
index 0a3fcb9..54a0d06 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionDecoratorImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionDecoratorImpl.java
@@ -28,7 +28,7 @@ import org.apache.geode.internal.admin.StatAlertDefinition;
  * all the statistic Threshold is valid for value evaluated by function
  * 
  */
-public final class FunctionDecoratorImpl extends BaseDecoratorImpl {
+public class FunctionDecoratorImpl extends BaseDecoratorImpl {
   private static final long serialVersionUID = -4857857489413081553L;
 
   protected short functorId = -1;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/GaugeThresholdDecoratorImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/GaugeThresholdDecoratorImpl.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/GaugeThresholdDecoratorImpl.java
index 3c68ab0..0a02001 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/GaugeThresholdDecoratorImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/GaugeThresholdDecoratorImpl.java
@@ -29,7 +29,7 @@ import org.apache.geode.internal.admin.StatAlertDefinition;
  * Implementation of {@link StatAlertDefinition}, represents threshold as data range
  * 
  */
-public final class GaugeThresholdDecoratorImpl extends BaseDecoratorImpl
+public class GaugeThresholdDecoratorImpl extends BaseDecoratorImpl
     implements DataSerializableFixedID {
 
   private static final long serialVersionUID = -8555077820685711783L;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/MultiAttrDefinitionImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/MultiAttrDefinitionImpl.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/MultiAttrDefinitionImpl.java
index c1414bb..506263c 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/MultiAttrDefinitionImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/MultiAttrDefinitionImpl.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.admin.StatAlertDefinition;
  * Implementation of {@link StatAlertDefinition} This provides the definition for multiple statistic
  * 
  */
-public final class MultiAttrDefinitionImpl implements StatAlertDefinition {
+public class MultiAttrDefinitionImpl implements StatAlertDefinition {
   private static final long serialVersionUID = 2508805676076940969L;
 
   protected String _name;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/NumberThresholdDecoratorImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/NumberThresholdDecoratorImpl.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/NumberThresholdDecoratorImpl.java
index 0abf142..c0d0f47 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/NumberThresholdDecoratorImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/NumberThresholdDecoratorImpl.java
@@ -29,7 +29,7 @@ import org.apache.geode.internal.admin.StatAlertDefinition;
  * Implementation of {@link StatAlertDefinition}, represents threshold as number
  * 
  */
-public final class NumberThresholdDecoratorImpl extends BaseDecoratorImpl
+public class NumberThresholdDecoratorImpl extends BaseDecoratorImpl
     implements DataSerializableFixedID {
 
   private static final long serialVersionUID = -1799140125261894306L;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/SingleAttrDefinitionImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/SingleAttrDefinitionImpl.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/SingleAttrDefinitionImpl.java
index 32222f0..50df697 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/SingleAttrDefinitionImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/SingleAttrDefinitionImpl.java
@@ -29,7 +29,7 @@ import org.apache.geode.internal.admin.StatAlertDefinition;
  * Implementation of {@link StatAlertDefinition} This provides the definition for single statistic
  * 
  */
-public final class SingleAttrDefinitionImpl implements StatAlertDefinition {
+public class SingleAttrDefinitionImpl implements StatAlertDefinition {
   private static final long serialVersionUID = 3292417185742697896L;
 
   protected String name;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/BucketAdvisor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/BucketAdvisor.java b/geode-core/src/main/java/org/apache/geode/internal/cache/BucketAdvisor.java
index 7b35cb5..f2640cd 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/BucketAdvisor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/BucketAdvisor.java
@@ -2441,7 +2441,7 @@ public class BucketAdvisor extends CacheDistributionAdvisor {
   /**
    * Profile information for a remote bucket hosted by cache servers.
    */
-  public static final class ServerBucketProfile extends BucketProfile {
+  public static class ServerBucketProfile extends BucketProfile {
 
     public Set<BucketServerLocation66> bucketServerLocations;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java b/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
index 62a9bc7..7ed1e3b 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
@@ -99,7 +99,7 @@ public class BucketRegion extends DistributedRegion implements Bucket {
    */
   private final AtomicLongWithTerminalState bytesInMemory = new AtomicLongWithTerminalState();
 
-  public static final class RawValue {
+  public static class RawValue {
     private final Object rawValue;
 
     public RawValue(Object rawVal) {

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/ClientRegionEventImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/ClientRegionEventImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/ClientRegionEventImpl.java
index 2fef594..30168ff 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/ClientRegionEventImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/ClientRegionEventImpl.java
@@ -32,7 +32,7 @@ import org.apache.geode.internal.cache.tier.sockets.ClientProxyMembershipID;
  * 
  * @since GemFire 5.1
  */
-public final class ClientRegionEventImpl extends RegionEventImpl {
+public class ClientRegionEventImpl extends RegionEventImpl {
 
   /**
    * The originating membershipId of this event.

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/CloseCacheMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/CloseCacheMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/CloseCacheMessage.java
index 03ef99b..6fe459e 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/CloseCacheMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/CloseCacheMessage.java
@@ -35,8 +35,7 @@ import org.apache.geode.internal.logging.LogService;
  */
 
 /** Creates a new instance of CloseCacheMessage */
-public final class CloseCacheMessage extends HighPriorityDistributionMessage
-    implements MessageWithReply {
+public class CloseCacheMessage extends HighPriorityDistributionMessage implements MessageWithReply {
   private static final Logger logger = LogService.getLogger();
 
   private int processorId;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/CreateRegionProcessor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/CreateRegionProcessor.java b/geode-core/src/main/java/org/apache/geode/internal/cache/CreateRegionProcessor.java
index 815b526..c1d1e77 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/CreateRegionProcessor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/CreateRegionProcessor.java
@@ -306,7 +306,7 @@ public class CreateRegionProcessor implements ProfileExchangeProcessor {
     }
   }
 
-  public static final class CreateRegionMessage extends HighPriorityDistributionMessage
+  public static class CreateRegionMessage extends HighPriorityDistributionMessage
       implements MessageWithReply {
 
     public boolean concurrencyChecksEnabled;
@@ -768,7 +768,7 @@ public class CreateRegionProcessor implements ProfileExchangeProcessor {
     }
   }
 
-  public static final class CreateRegionReplyMessage extends ReplyMessage {
+  public static class CreateRegionReplyMessage extends ReplyMessage {
     protected CacheProfile profile;
     protected ArrayList bucketProfiles;
     protected Object eventState;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyOperation.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyOperation.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyOperation.java
index 5132ec0..ad3f976 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyOperation.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyOperation.java
@@ -217,7 +217,7 @@ public class DestroyOperation extends DistributedCacheOperation {
     }
   }
 
-  public static final class DestroyWithContextMessage extends DestroyMessage {
+  public static class DestroyWithContextMessage extends DestroyMessage {
     transient ClientProxyMembershipID context;
 
     public DestroyWithContextMessage() {}

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyPartitionedRegionMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyPartitionedRegionMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyPartitionedRegionMessage.java
index 5914ab5..0af8840 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyPartitionedRegionMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyPartitionedRegionMessage.java
@@ -57,7 +57,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
  * 
  * @since GemFire 5.0
  */
-public final class DestroyPartitionedRegionMessage extends PartitionMessage {
+public class DestroyPartitionedRegionMessage extends PartitionMessage {
   private static final Logger logger = LogService.getLogger();
 
   private Object cbArg;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyRegionOperation.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyRegionOperation.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyRegionOperation.java
index 3cc988f..68b1ae8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyRegionOperation.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DestroyRegionOperation.java
@@ -466,7 +466,7 @@ public class DestroyRegionOperation extends DistributedCacheOperation {
     }
   }
 
-  public static final class DestroyRegionWithContextMessage extends DestroyRegionMessage {
+  public static class DestroyRegionWithContextMessage extends DestroyRegionMessage {
 
     protected transient Object context;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DiskWriteAttributesImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DiskWriteAttributesImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DiskWriteAttributesImpl.java
index a129eda..d5893d4 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DiskWriteAttributesImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DiskWriteAttributesImpl.java
@@ -34,7 +34,7 @@ import java.util.Properties;
  * @since GemFire 5.1
  */
 @SuppressWarnings({"deprecation", "unused"})
-public final class DiskWriteAttributesImpl implements DiskWriteAttributes {
+public class DiskWriteAttributesImpl implements DiskWriteAttributes {
   private static final long serialVersionUID = -4269181954992768424L;
 
   /** Are writes synchronous? */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistPeerTXStateStub.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistPeerTXStateStub.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistPeerTXStateStub.java
index be454e7..a2eeaed 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistPeerTXStateStub.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistPeerTXStateStub.java
@@ -39,8 +39,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
  * 
  *
  */
-public final class DistPeerTXStateStub extends PeerTXStateStub
-    implements DistTXCoordinatorInterface {
+public class DistPeerTXStateStub extends PeerTXStateStub implements DistTXCoordinatorInterface {
   private ArrayList<DistTxEntryEvent> primaryTransactionalOperations = null;
   private ArrayList<DistTxEntryEvent> secondaryTransactionalOperations = null;
   private DistTXPrecommitMessage precommitDistTxMsg = null;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXCommitMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXCommitMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXCommitMessage.java
index e22e1d9..be401b9 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXCommitMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXCommitMessage.java
@@ -171,7 +171,7 @@ public class DistTXCommitMessage extends TXMessage {
    * stub to the tx host. This is the reply to a {@link DistTXCommitMessage}.
    * 
    */
-  public static final class DistTXCommitReplyMessage extends ReplyMessage {
+  public static class DistTXCommitReplyMessage extends ReplyMessage {
     private transient TXCommitMessage commitMessage;
 
     /**
@@ -274,7 +274,7 @@ public class DistTXCommitMessage extends TXMessage {
    * 
    *      [DISTTX] TODO see if need ReliableReplyProcessor21? departed members?
    */
-  public static final class DistTxCommitReplyProcessor extends ReplyProcessor21 {
+  public static class DistTxCommitReplyProcessor extends ReplyProcessor21 {
     private HashMap<DistributedMember, DistTXCoordinatorInterface> msgMap;
     private Map<DistributedMember, TXCommitMessage> commitResponseMap;
     private transient TXId txIdent = null;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXPrecommitMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXPrecommitMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXPrecommitMessage.java
index 0ab2cc3..88e84f8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXPrecommitMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXPrecommitMessage.java
@@ -56,7 +56,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
 /**
  *
  */
-public final class DistTXPrecommitMessage extends TXMessage {
+public class DistTXPrecommitMessage extends TXMessage {
 
   private static final Logger logger = LogService.getLogger();
   ArrayList<DistTxEntryEvent> secondaryTransactionalOperations;
@@ -175,7 +175,7 @@ public final class DistTXPrecommitMessage extends TXMessage {
   /**
    * This is the reply to a {@link DistTXPrecommitMessage}.
    */
-  public static final class DistTXPrecommitReplyMessage extends ReplyMessage {
+  public static class DistTXPrecommitReplyMessage extends ReplyMessage {
     private transient DistTxPrecommitResponse commitResponse;
 
     /**
@@ -277,7 +277,7 @@ public final class DistTXPrecommitMessage extends TXMessage {
    *
    *      [DISTTX] TODO see if need ReliableReplyProcessor21? departed members?
    */
-  public static final class DistTxPrecommitReplyProcessor extends ReplyProcessor21 {
+  public static class DistTxPrecommitReplyProcessor extends ReplyProcessor21 {
     private HashMap<DistributedMember, DistTXCoordinatorInterface> msgMap;
     private Map<DistributedMember, DistTxPrecommitResponse> commitResponseMap;
     private transient TXId txIdent = null;
@@ -465,7 +465,7 @@ public final class DistTXPrecommitMessage extends TXMessage {
     }
   }
 
-  public static final class DistTxPrecommitResponse implements DataSerializableFixedID {
+  public static class DistTxPrecommitResponse implements DataSerializableFixedID {
     private transient Boolean commitState;
     private transient ArrayList<ArrayList<DistTxThinEntryState>> distTxEventList;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXRollbackMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXRollbackMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXRollbackMessage.java
index d4f5943..9438fc2 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXRollbackMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXRollbackMessage.java
@@ -52,7 +52,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
 /**
  * 
  */
-public final class DistTXRollbackMessage extends TXMessage {
+public class DistTXRollbackMessage extends TXMessage {
 
   private static final Logger logger = LogService.getLogger();
 
@@ -132,7 +132,7 @@ public final class DistTXRollbackMessage extends TXMessage {
   /**
    * This is the reply to a {@link DistTXRollbackMessage}.
    */
-  public static final class DistTXRollbackReplyMessage extends ReplyMessage {
+  public static class DistTXRollbackReplyMessage extends ReplyMessage {
     private transient Boolean rollbackState;
 
     /**
@@ -283,7 +283,7 @@ public final class DistTXRollbackMessage extends TXMessage {
    * 
    *      [DISTTX] TODO see if need ReliableReplyProcessor21? departed members?
    */
-  public static final class DistTxRollbackReplyProcessor extends ReplyProcessor21 {
+  public static class DistTxRollbackReplyProcessor extends ReplyProcessor21 {
     private HashMap<DistributedMember, DistTXCoordinatorInterface> msgMap;
     private Map<DistributedMember, Boolean> rollbackResponseMap;
     private transient TXId txIdent = null;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXStateOnCoordinator.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXStateOnCoordinator.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXStateOnCoordinator.java
index a21e505..ca5764a 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXStateOnCoordinator.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistTXStateOnCoordinator.java
@@ -31,8 +31,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
  * 
  * 
  */
-public final class DistTXStateOnCoordinator extends DistTXState
-    implements DistTXCoordinatorInterface {
+public class DistTXStateOnCoordinator extends DistTXState implements DistTXCoordinatorInterface {
 
   private ArrayList<DistTxEntryEvent> primaryTransactionalOperations = null;
   private ArrayList<DistTxEntryEvent> secondaryTransactionalOperations = null;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedClearOperation.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedClearOperation.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedClearOperation.java
index 9d10fc1..9f26cc9 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedClearOperation.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedClearOperation.java
@@ -262,7 +262,7 @@ public class DistributedClearOperation extends DistributedCacheOperation {
 
   }
 
-  public static final class ClearRegionWithContextMessage extends ClearRegionMessage {
+  public static class ClearRegionWithContextMessage extends ClearRegionMessage {
     protected transient Object context;
     protected RegionVersionVector rvv;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedPutAllOperation.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedPutAllOperation.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedPutAllOperation.java
index 61542c4..7e76981 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedPutAllOperation.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedPutAllOperation.java
@@ -268,7 +268,7 @@ public class DistributedPutAllOperation extends AbstractUpdateOperation {
   /**
    * Data that represents a single entry being putall'd.
    */
-  public static final class PutAllEntryData {
+  public static class PutAllEntryData {
 
     final Object key;
 
@@ -589,7 +589,7 @@ public class DistributedPutAllOperation extends AbstractUpdateOperation {
     }
   }
 
-  public static final class EntryVersionsList extends ArrayList<VersionTag>
+  public static class EntryVersionsList extends ArrayList<VersionTag>
       implements DataSerializableFixedID, Externalizable {
 
     public EntryVersionsList() {

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
index 41b2613..e20a2c4 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
@@ -3544,7 +3544,7 @@ public class DistributedRegion extends LocalRegion implements CacheDistributionA
    * Implementation of {@link ProfileVisitor} that selects a random replicated member from the
    * available ones for this region.
    */
-  static final class GetRandomReplicate implements ProfileVisitor<DistributedMember> {
+  static class GetRandomReplicate implements ProfileVisitor<DistributedMember> {
 
     private boolean onlyPersistent = false;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRemoveAllOperation.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRemoveAllOperation.java b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRemoveAllOperation.java
index 0c13b59..5a1b1be 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRemoveAllOperation.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRemoveAllOperation.java
@@ -254,7 +254,7 @@ public class DistributedRemoveAllOperation extends AbstractUpdateOperation // TO
   /**
    * Data that represents a single entry being RemoveAll'd.
    */
-  public static final class RemoveAllEntryData {
+  public static class RemoveAllEntryData {
 
     final Object key;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java
index ac4954a..3cb5bbd 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java
@@ -2525,7 +2525,7 @@ public class EntryEventImpl
     return result;
   }
 
-  public static final class SerializedCacheValueImpl
+  public static class SerializedCacheValueImpl
       implements SerializedCacheValue, CachedDeserializable, Sendable {
     private final EntryEventImpl event;
     @Unretained

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/EvictionAttributesImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/EvictionAttributesImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/EvictionAttributesImpl.java
index 33e4895..7d90bef 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/EvictionAttributesImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/EvictionAttributesImpl.java
@@ -47,7 +47,7 @@ import org.apache.geode.internal.cache.lru.MemLRUCapacityController;
  * 
  * @since GemFire 5.0
  */
-public final class EvictionAttributesImpl extends EvictionAttributes
+public class EvictionAttributesImpl extends EvictionAttributes
     implements EvictionAttributesMutator {
   private static final long serialVersionUID = -6404395520499379715L;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/GridAdvisor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/GridAdvisor.java b/geode-core/src/main/java/org/apache/geode/internal/cache/GridAdvisor.java
index 03c14ab..c21667d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/GridAdvisor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/GridAdvisor.java
@@ -379,7 +379,7 @@ public abstract class GridAdvisor extends DistributionAdvisor {
   /**
    * Basic idea is to base id off of host and port of GridProfile
    */
-  public static final class GridProfileId implements ProfileId {
+  public static class GridProfileId implements ProfileId {
 
     private final GridProfile gp;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/HARegion.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/HARegion.java b/geode-core/src/main/java/org/apache/geode/internal/cache/HARegion.java
index f80f971..e1d3d6b 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/HARegion.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/HARegion.java
@@ -54,7 +54,7 @@ import org.apache.geode.internal.offheap.annotations.Released;
  * @since GemFire 4.3
  * 
  */
-public final class HARegion extends DistributedRegion {
+public class HARegion extends DistributedRegion {
   private static final Logger logger = LogService.getLogger();
 
   CachePerfStats haRegionStats;