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:09 UTC

[2/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/cache/partitioned/PrimaryRequestMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PrimaryRequestMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PrimaryRequestMessage.java
index 75bd0b6..f053d5f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PrimaryRequestMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PrimaryRequestMessage.java
@@ -46,7 +46,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
  * 
  *
  */
-public final class PrimaryRequestMessage extends PartitionMessage {
+public class PrimaryRequestMessage extends PartitionMessage {
   private static final Logger logger = LogService.getLogger();
 
   private static final long serialVersionUID = 1L;
@@ -136,7 +136,7 @@ public final class PrimaryRequestMessage extends PartitionMessage {
   /**
    * The reply to a PrimarRequestMessage, indicating if the sender is the primary
    */
-  static public final class PrimaryRequestReplyMessage extends ReplyMessage {
+  static public class PrimaryRequestReplyMessage extends ReplyMessage {
     private static final long serialVersionUID = 1L;
 
     public volatile boolean isPrimary;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PutMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PutMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PutMessage.java
index 1c41e61..25c1d00 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PutMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PutMessage.java
@@ -75,7 +75,7 @@ import static org.apache.geode.internal.offheap.annotations.OffHeapIdentifier.EN
  *
  * @since GemFire 5.0
  */
-public final class PutMessage extends PartitionMessageWithDirectReply implements NewValueImporter {
+public class PutMessage extends PartitionMessageWithDirectReply implements NewValueImporter {
   private static final Logger logger = LogService.getLogger();
 
   /** The key associated with the value that must be sent */
@@ -877,7 +877,7 @@ public final class PutMessage extends PartitionMessageWithDirectReply implements
     return _mayAddToMultipleSerialGateways(dm);
   }
 
-  public static final class PutReplyMessage extends ReplyMessage implements OldValueImporter {
+  public static class PutReplyMessage extends ReplyMessage implements OldValueImporter {
     /** Result of the Put operation */
     boolean result;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/QueryMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/QueryMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/QueryMessage.java
index b6028fe..bbd9d9c 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/QueryMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/QueryMessage.java
@@ -54,7 +54,7 @@ import org.apache.geode.internal.logging.LogService;
 import org.apache.geode.internal.logging.log4j.LogMarker;
 import org.apache.geode.cache.query.Struct;
 
-public final class QueryMessage extends StreamingPartitionOperation.StreamingPartitionMessage {
+public class QueryMessage extends StreamingPartitionOperation.StreamingPartitionMessage {
   private static final Logger logger = LogService.getLogger();
 
   private volatile String queryString;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RegionAdvisor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RegionAdvisor.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RegionAdvisor.java
index 8f4c65c..dc3efff 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RegionAdvisor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RegionAdvisor.java
@@ -515,7 +515,7 @@ public class RegionAdvisor extends CacheDistributionAdvisor {
   /**
    * Profile information for a remote counterpart.
    */
-  public static final class PartitionProfile extends CacheProfile {
+  public static class PartitionProfile extends CacheProfile {
 
     /**
      * The number of Mb the VM is allowed to use for the PR

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoteSizeMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoteSizeMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoteSizeMessage.java
index 6e4acff..5595cb2 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoteSizeMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoteSizeMessage.java
@@ -45,7 +45,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
  * 
  * @since GemFire 5.0
  */
-public final class RemoteSizeMessage extends RemoteOperationMessage {
+public class RemoteSizeMessage extends RemoteOperationMessage {
   private static final Logger logger = LogService.getLogger();
 
   /** query type for Entries */
@@ -179,7 +179,7 @@ public final class RemoteSizeMessage extends RemoteOperationMessage {
     out.writeByte((byte) queryType);
   }
 
-  public static final class SizeReplyMessage extends ReplyMessage {
+  public static class SizeReplyMessage extends ReplyMessage {
     /** Propagated exception from remote node to operation initiator */
     private int size;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveAllPRMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveAllPRMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveAllPRMessage.java
index 5f05e4b..4ef4157 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveAllPRMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveAllPRMessage.java
@@ -77,7 +77,7 @@ import org.apache.geode.internal.offheap.annotations.Retained;
  *
  * @since GemFire 8.1
  */
-public final class RemoveAllPRMessage extends PartitionMessageWithDirectReply {
+public class RemoveAllPRMessage extends PartitionMessageWithDirectReply {
   private static final Logger logger = LogService.getLogger();
 
   private RemoveAllEntryData[] removeAllPRData;
@@ -664,7 +664,7 @@ public final class RemoveAllPRMessage extends PartitionMessageWithDirectReply {
     return _mayAddToMultipleSerialGateways(dm);
   }
 
-  public static final class RemoveAllReplyMessage extends ReplyMessage {
+  public static class RemoveAllReplyMessage extends ReplyMessage {
     /** Result of the RemoveAll operation */
     boolean result;
     VersionedObjectList versions;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveBucketMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveBucketMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveBucketMessage.java
index 6336841..a8ff068 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveBucketMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveBucketMessage.java
@@ -139,7 +139,7 @@ public class RemoveBucketMessage extends PartitionMessage {
     out.writeBoolean(this.forceRemovePrimary);
   }
 
-  public static final class RemoveBucketReplyMessage extends ReplyMessage {
+  public static class RemoveBucketReplyMessage extends ReplyMessage {
 
     private boolean removed;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveIndexesMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveIndexesMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveIndexesMessage.java
index 17b5f71..5aff6da 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveIndexesMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveIndexesMessage.java
@@ -49,7 +49,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
  * 
  * 
  */
-public final class RemoveIndexesMessage extends PartitionMessage {
+public class RemoveIndexesMessage extends PartitionMessage {
   private static final Logger logger = LogService.getLogger();
 
   /**
@@ -413,7 +413,7 @@ public final class RemoveIndexesMessage extends PartitionMessage {
    * index creation.
    *
    */
-  public static final class RemoveIndexesReplyMessage extends ReplyMessage {
+  public static class RemoveIndexesReplyMessage extends ReplyMessage {
 
     /** Indexes removed or not. */
     private boolean result;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/SizeMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/SizeMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/SizeMessage.java
index 3dc5217..0c6aea8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/SizeMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/SizeMessage.java
@@ -50,7 +50,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
  * 
  * @since GemFire 5.0
  */
-public final class SizeMessage extends PartitionMessage {
+public class SizeMessage extends PartitionMessage {
   private static final Logger logger = LogService.getLogger();
 
   /** The list of buckets whose size is needed, if null, then all buckets */
@@ -206,7 +206,7 @@ public final class SizeMessage extends PartitionMessage {
     DataSerializer.writeArrayList(this.bucketIds, out);
   }
 
-  public static final class SizeReplyMessage extends ReplyMessage {
+  public static class SizeReplyMessage extends ReplyMessage {
     /** Propagated exception from remote node to operation initiator */
     private Map<Integer, SizeEntry> bucketSizes;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java
index 73b8ba9..cfb5fd4 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java
@@ -394,7 +394,7 @@ public class RegionSnapshotServiceImpl<K, V> implements RegionSnapshotService<K,
    * Writes snapshot data to a {@link SnapshotWriter}. Caller is responsible for invoking
    * {@link SnapshotWriter#snapshotComplete()}.
    */
-  static final class SnapshotWriterSink implements ExportSink {
+  static class SnapshotWriterSink implements ExportSink {
     private final SnapshotWriter writer;
     private long bytes;
 
@@ -419,7 +419,7 @@ public class RegionSnapshotServiceImpl<K, V> implements RegionSnapshotService<K,
    * Forwards snapshot data to a {@link ResultSender}. Caller is responsible for invoking
    * {@link ResultSender#lastResult(Object)}.
    */
-  static final class ResultSenderSink implements ExportSink {
+  static class ResultSenderSink implements ExportSink {
     /** the fowarding destination */
     private final ResultSender<SnapshotRecord[]> sender;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/tier/InternalClientMembership.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/InternalClientMembership.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/InternalClientMembership.java
index 7b291da..504081d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/InternalClientMembership.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/InternalClientMembership.java
@@ -58,7 +58,7 @@ import org.apache.geode.management.membership.ClientMembershipListener;
  *
  * @since GemFire 4.2.1
  */
-public final class InternalClientMembership {
+public class InternalClientMembership {
 
   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/cache/tier/sockets/ClientInstantiatorMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
index ef3b594..4b6d581 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientInstantiatorMessage.java
@@ -34,7 +34,7 @@ import org.apache.geode.internal.cache.tier.MessageType;
  * 
  * @since GemFire 5.0
  */
-public final class ClientInstantiatorMessage extends ClientUpdateMessageImpl {
+public class ClientInstantiatorMessage extends ClientUpdateMessageImpl {
   private static final long serialVersionUID = 2949326125521840437L;
   /**
    * Serialized 2D array of the instantiators

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientMarkerMessageImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientMarkerMessageImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientMarkerMessageImpl.java
index 4e29262..5aedcef 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientMarkerMessageImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientMarkerMessageImpl.java
@@ -31,7 +31,7 @@ import java.io.*;
  *
  * @since GemFire 5.5
  */
-public final class ClientMarkerMessageImpl implements ClientMessage {
+public class ClientMarkerMessageImpl implements ClientMessage {
   private static final long serialVersionUID = 5423895238521508743L;
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientPingMessageImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientPingMessageImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientPingMessageImpl.java
index 324fcb8..a98214c 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientPingMessageImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientPingMessageImpl.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.cache.tier.MessageType;
  * 
  * @since GemFire 6.6.2.x
  */
-public final class ClientPingMessageImpl implements ClientMessage {
+public class ClientPingMessageImpl implements ClientMessage {
 
   private static final long serialVersionUID = 5423895238521508743L;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/InterestResultPolicyImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/InterestResultPolicyImpl.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/InterestResultPolicyImpl.java
index 8acb558..bfc3937 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/InterestResultPolicyImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/InterestResultPolicyImpl.java
@@ -28,7 +28,7 @@ import java.io.*;
  *
  * @since GemFire 5.7
  */
-public final class InterestResultPolicyImpl extends InterestResultPolicy
+public class InterestResultPolicyImpl extends InterestResultPolicy
     implements DataSerializableFixedID {
   private static final long serialVersionUID = -7456596794818237831L;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java
index 6eadee3..83d0e9d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java
@@ -669,7 +669,7 @@ public class ServerConnection implements Runnable {
     }
   }
 
-  static final class Counter {
+  static class Counter {
     int cnt;
 
     void incr() {

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
index 9472792..72ad77b 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderAdvisor.java
@@ -488,7 +488,7 @@ public class GatewaySenderAdvisor extends DistributionAdvisor {
   /**
    * Profile information for a remote counterpart.
    */
-  public static final class GatewaySenderProfile extends DistributionAdvisor.Profile {
+  public static class GatewaySenderProfile extends DistributionAdvisor.Profile {
     public String Id;
 
     public long startTime;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventCallbackArgument.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventCallbackArgument.java b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventCallbackArgument.java
index 4a3f398..d944997 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventCallbackArgument.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventCallbackArgument.java
@@ -41,7 +41,7 @@ import java.util.Set;
  * 
  * @since GemFire 7.0
  */
-public final class GatewaySenderEventCallbackArgument extends WrappedCallbackArgument
+public class GatewaySenderEventCallbackArgument extends WrappedCallbackArgument
     implements DataSerializableFixedID {
   /**
    * The id of the originating <code>GatewayReceiver</code> making the request

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ParallelQueueBatchRemovalMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ParallelQueueBatchRemovalMessage.java b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ParallelQueueBatchRemovalMessage.java
index 47005c3..cd0ba30 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ParallelQueueBatchRemovalMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ParallelQueueBatchRemovalMessage.java
@@ -154,7 +154,7 @@ public class ParallelQueueBatchRemovalMessage extends PartitionMessage {
     this.bucketToTailKey = (Map) DataSerializer.readObject(in);
   }
 
-  public static final class BatchRemovalReplyMessage extends ReplyMessage {
+  public static class BatchRemovalReplyMessage extends ReplyMessage {
     /**
      * Empty constructor to conform to DataSerializable interface
      */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/GeodeEntityResolver.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/GeodeEntityResolver.java b/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/GeodeEntityResolver.java
index 194b2ec..cdd9659 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/GeodeEntityResolver.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/GeodeEntityResolver.java
@@ -32,7 +32,7 @@ import org.xml.sax.ext.EntityResolver2;
  * 
  * @since GemFire 8.1
  */
-public final class GeodeEntityResolver extends DefaultEntityResolver2 {
+public class GeodeEntityResolver extends DefaultEntityResolver2 {
 
   private static final String SYSTEM_ID_ROOT = "http://geode.apache.org/";
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/PivotalEntityResolver.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/PivotalEntityResolver.java b/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/PivotalEntityResolver.java
index 48ac881..68e43dc 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/PivotalEntityResolver.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/PivotalEntityResolver.java
@@ -32,7 +32,7 @@ import org.xml.sax.ext.EntityResolver2;
  * 
  * @since GemFire 8.1
  */
-public final class PivotalEntityResolver extends DefaultEntityResolver2 {
+public class PivotalEntityResolver extends DefaultEntityResolver2 {
 
   private static final String SYSTEM_ID_ROOT = "http://schema.pivotal.io/";
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/XmlGeneratorUtils.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/XmlGeneratorUtils.java b/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/XmlGeneratorUtils.java
index 3959316..5a4a052 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/XmlGeneratorUtils.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/XmlGeneratorUtils.java
@@ -29,7 +29,7 @@ import org.xml.sax.helpers.AttributesImpl;
  * @since GemFire 8.1
  */
 // UnitTest XmlGeneratorUtilsTest
-public final class XmlGeneratorUtils {
+public class XmlGeneratorUtils {
 
   private XmlGeneratorUtils() {
     // statics only

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/concurrent/AtomicLong5.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/concurrent/AtomicLong5.java b/geode-core/src/main/java/org/apache/geode/internal/concurrent/AtomicLong5.java
index f2d21fd..89ad5ea 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/concurrent/AtomicLong5.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/concurrent/AtomicLong5.java
@@ -17,7 +17,7 @@ package org.apache.geode.internal.concurrent;
 /**
  * AL implementation for JDK 5.
  */
-public final class AtomicLong5 extends java.util.concurrent.atomic.AtomicLong implements AL {
+public class AtomicLong5 extends java.util.concurrent.atomic.AtomicLong implements AL {
   private static final long serialVersionUID = -1915700199064062938L;
 
   public AtomicLong5() {

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/concurrent/CompactConcurrentHashSet2.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/concurrent/CompactConcurrentHashSet2.java b/geode-core/src/main/java/org/apache/geode/internal/concurrent/CompactConcurrentHashSet2.java
index 1bc4add..d184f98 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/concurrent/CompactConcurrentHashSet2.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/concurrent/CompactConcurrentHashSet2.java
@@ -1030,7 +1030,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
   /**
    * A node inserted at head of bins during transfer operations.
    */
-  static final class ForwardingNode<K> extends Node<K> {
+  static class ForwardingNode<K> extends Node<K> {
     final Node<K>[] nextTable;
 
     ForwardingNode(Node<K>[] tab) {
@@ -1060,7 +1060,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
   /**
    * A place-holder node used in computeIfAbsent and compute
    */
-  static final class ReservationNode<K> extends Node<K> {
+  static class ReservationNode<K> extends Node<K> {
     ReservationNode() {
       super(RESERVED, null, null);
     }
@@ -1400,7 +1400,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
   /**
    * Nodes for use in TreeBins
    */
-  static final class TreeNode<K> extends Node<K> {
+  static class TreeNode<K> extends Node<K> {
     TreeNode<K> parent; // red-black tree links
     TreeNode<K> left;
     TreeNode<K> right;
@@ -1459,7 +1459,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
    * forcing writers (who hold bin lock) to wait for readers (who do not) to complete before tree
    * restructuring operations.
    */
-  static final class TreeBin<K> extends Node<K> {
+  static class TreeBin<K> extends Node<K> {
     TreeNode<K> root;
     volatile TreeNode<K> first;
     volatile Thread waiter;
@@ -1979,7 +1979,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
    * Records the table, its length, and current traversal index for a traverser that must process a
    * region of a forwarded table before proceeding with current table.
    */
-  static final class TableStack<K> {
+  static class TableStack<K> {
     int length;
     int index;
     Node<K>[] tab;
@@ -2120,7 +2120,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
     }
   }
 
-  static final class KeyIterator<K> extends BaseIterator<K> implements Iterator<K>, Enumeration<K> {
+  static class KeyIterator<K> extends BaseIterator<K> implements Iterator<K>, Enumeration<K> {
     KeyIterator(Node<K>[] tab, int index, int size, int limit, CompactConcurrentHashSet2<K> map) {
       super(tab, index, size, limit, map);
     }
@@ -2147,7 +2147,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
   // See their internal docs for explanation.
 
   // A padded cell for distributing counts
-  static final class CounterCell {
+  static class CounterCell {
     volatile long p0, p1, p2, p3, p4, p5, p6;
     volatile long value;
     volatile long q0, q1, q2, q3, q4, q5, q6;
@@ -2161,7 +2161,7 @@ public class CompactConcurrentHashSet2<V> extends AbstractSet<V> implements Set<
    * Holder for the thread-local hash code determining which CounterCell to use. The code is
    * initialized via the counterHashCodeGenerator, but may be moved upon collisions.
    */
-  static final class CounterHashCode {
+  static class CounterHashCode {
     int code;
   }
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/DateFormatter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/DateFormatter.java b/geode-core/src/main/java/org/apache/geode/internal/logging/DateFormatter.java
index 7f53a23..4fa45e6 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/DateFormatter.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/DateFormatter.java
@@ -22,7 +22,7 @@ import java.util.Date;
  * Defines the common date format for GemFire and provides DateFormat instances.
  * 
  */
-public final class DateFormatter {
+public class DateFormatter {
 
   /**
    * The format string used to format the timestamp of GemFire log messages

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/GemFireHandler.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/GemFireHandler.java b/geode-core/src/main/java/org/apache/geode/internal/logging/GemFireHandler.java
index 324f464..bef89be 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/GemFireHandler.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/GemFireHandler.java
@@ -26,7 +26,7 @@ import org.apache.geode.internal.logging.log4j.LogWriterLogger;
  * Implementation of the standard JDK handler that publishes a log record to a LogWriterImpl. Note
  * this handler ignores any installed handler.
  */
-public final class GemFireHandler extends Handler {
+public class GemFireHandler extends Handler {
 
   /**
    * Use the log writer to use some of its formatting code.

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityLogWriter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityLogWriter.java b/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityLogWriter.java
index c95a375..d791835 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityLogWriter.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityLogWriter.java
@@ -28,7 +28,7 @@ import org.apache.geode.i18n.StringId;
  * 
  * @since GemFire 5.5
  */
-public final class SecurityLogWriter extends LogWriterImpl implements InternalLogWriter {
+public class SecurityLogWriter extends LogWriterImpl implements InternalLogWriter {
 
   public static final String SECURITY_PREFIX = DistributionConfig.SECURITY_PREFIX_NAME;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityManagerLogWriter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityManagerLogWriter.java b/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityManagerLogWriter.java
index 3e62f7d..d0c6eaa 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityManagerLogWriter.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/SecurityManagerLogWriter.java
@@ -29,7 +29,7 @@ import java.io.PrintStream;
  * 
  * @since GemFire 5.5
  */
-public final class SecurityManagerLogWriter extends ManagerLogWriter {
+public class SecurityManagerLogWriter extends ManagerLogWriter {
 
   public SecurityManagerLogWriter(int level, PrintStream stream) {
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/AlertAppender.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/AlertAppender.java b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/AlertAppender.java
index 8a50050..a7151cc 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/AlertAppender.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/AlertAppender.java
@@ -42,7 +42,7 @@ import org.apache.geode.internal.tcp.ReenteredConnectException;
  * to the log file.
  * 
  */
-public final class AlertAppender extends AbstractAppender implements PropertyChangeListener {
+public class AlertAppender extends AbstractAppender implements PropertyChangeListener {
   private static final String APPENDER_NAME = AlertAppender.class.getName();
   private static final Logger logger = LogService.getLogger();
   private static final AlertAppender instance = createAlertAppender();

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LocalizedMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LocalizedMessage.java b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LocalizedMessage.java
index 3364759..870ca4f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LocalizedMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LocalizedMessage.java
@@ -23,7 +23,7 @@ import org.apache.geode.i18n.StringId;
  * An implementation of a Log4j {@link Message} that wraps a {@link StringId}.
  * 
  */
-public final class LocalizedMessage implements Message {
+public class LocalizedMessage implements Message {
   private static final long serialVersionUID = -8893339995741536401L;
 
   private final StringId stringId;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LogWriterLogger.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LogWriterLogger.java b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LogWriterLogger.java
index 2a248cc..fad126b 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LogWriterLogger.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/LogWriterLogger.java
@@ -38,7 +38,7 @@ import java.util.logging.Handler;
  * 
  */
 @SuppressWarnings("unused")
-public final class LogWriterLogger extends FastLogger implements InternalLogWriter, GemFireLogger {
+public class LogWriterLogger extends FastLogger implements InternalLogWriter, GemFireLogger {
 
   private static final long serialVersionUID = 446081244292135L;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/ThreadIdPatternConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/ThreadIdPatternConverter.java b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/ThreadIdPatternConverter.java
index 730fc04..338b1d6 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/ThreadIdPatternConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/ThreadIdPatternConverter.java
@@ -24,7 +24,7 @@ import org.apache.logging.log4j.core.pattern.LogEventPatternConverter;
  */
 @Plugin(name = "ThreadIdPatternConverter", category = "Converter")
 @ConverterKeys({"tid", "threadId"})
-public final class ThreadIdPatternConverter extends LogEventPatternConverter {
+public class ThreadIdPatternConverter extends LogEventPatternConverter {
   /**
    * Singleton.
    */

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/message/GemFireParameterizedMessageFactory.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/message/GemFireParameterizedMessageFactory.java b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/message/GemFireParameterizedMessageFactory.java
index 09f2899..f0d40f2 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/message/GemFireParameterizedMessageFactory.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/logging/log4j/message/GemFireParameterizedMessageFactory.java
@@ -32,7 +32,7 @@ import org.apache.logging.log4j.message.ParameterizedMessage;
  * Geode changes include changing class name and package. Additional changes are commented with "//
  * GEODE: note"
  */
-public final class GemFireParameterizedMessageFactory extends AbstractMessageFactory {
+public class GemFireParameterizedMessageFactory extends AbstractMessageFactory {
 
   private static final long serialVersionUID = 1L;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java b/geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java
index 82e36ab..fec81ca 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java
@@ -607,7 +607,7 @@ public class SocketCreator {
     return extendedKeyManagers;
   }
 
-  private static final class ExtendedAliasKeyManager extends X509ExtendedKeyManager {
+  private static class ExtendedAliasKeyManager extends X509ExtendedKeyManager {
 
     private final X509ExtendedKeyManager delegate;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/offheap/FreeListManager.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/offheap/FreeListManager.java b/geode-core/src/main/java/org/apache/geode/internal/offheap/FreeListManager.java
index 670f789..ad9fa09 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/offheap/FreeListManager.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/offheap/FreeListManager.java
@@ -883,7 +883,7 @@ public class FreeListManager {
   /**
    * Used to represent an address from a tiny free list as a MemoryBlock
    */
-  protected static final class TinyMemoryBlock implements MemoryBlock {
+  protected static class TinyMemoryBlock implements MemoryBlock {
     private final long address;
     private final int freeListId;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/AttachProcessUtils.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/AttachProcessUtils.java b/geode-core/src/main/java/org/apache/geode/internal/process/AttachProcessUtils.java
index b3e9ba6..2103785 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/AttachProcessUtils.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/AttachProcessUtils.java
@@ -23,7 +23,7 @@ import com.sun.tools.attach.VirtualMachineDescriptor;
  * 
  * @since GemFire 8.0
  */
-final class AttachProcessUtils implements InternalProcessUtils {
+class AttachProcessUtils implements InternalProcessUtils {
 
   AttachProcessUtils() {}
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/BlockingProcessStreamReader.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/BlockingProcessStreamReader.java b/geode-core/src/main/java/org/apache/geode/internal/process/BlockingProcessStreamReader.java
index 7a47b1e..9d2b497 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/BlockingProcessStreamReader.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/BlockingProcessStreamReader.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.logging.LogService;
  * 
  * @since GemFire 8.2
  */
-public final class BlockingProcessStreamReader extends ProcessStreamReader {
+public class BlockingProcessStreamReader extends ProcessStreamReader {
   private static final Logger logger = LogService.getLogger();
 
   protected BlockingProcessStreamReader(final Builder builder) {

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ClusterConfigurationNotAvailableException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ClusterConfigurationNotAvailableException.java b/geode-core/src/main/java/org/apache/geode/internal/process/ClusterConfigurationNotAvailableException.java
index bb4fba2..7699cc4 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ClusterConfigurationNotAvailableException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ClusterConfigurationNotAvailableException.java
@@ -20,7 +20,7 @@ package org.apache.geode.internal.process;
  * 
  * @since GemFire 8.0
  */
-public final class ClusterConfigurationNotAvailableException extends RuntimeException {
+public class ClusterConfigurationNotAvailableException extends RuntimeException {
   private static final long serialVersionUID = -3448160213553925462L;
 
   public ClusterConfigurationNotAvailableException(String message) {

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ConnectionFailedException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ConnectionFailedException.java b/geode-core/src/main/java/org/apache/geode/internal/process/ConnectionFailedException.java
index 7765ef3..0c04130 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ConnectionFailedException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ConnectionFailedException.java
@@ -19,7 +19,7 @@ package org.apache.geode.internal.process;
  * 
  * @since GemFire 7.0
  */
-public final class ConnectionFailedException extends Exception {
+public class ConnectionFailedException extends Exception {
   private static final long serialVersionUID = 5622636452836752700L;
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ControlFileWatchdog.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ControlFileWatchdog.java b/geode-core/src/main/java/org/apache/geode/internal/process/ControlFileWatchdog.java
index beae4b4..0af8758 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ControlFileWatchdog.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ControlFileWatchdog.java
@@ -26,7 +26,7 @@ import org.apache.geode.internal.logging.LogService;
  * 
  * @since GemFire 8.0
  */
-final class ControlFileWatchdog implements Runnable {
+class ControlFileWatchdog implements Runnable {
   private static final Logger logger = LogService.getLogger();
 
   private static final long STOP_TIMEOUT_MILLIS = 60 * 1000;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ControllableProcess.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ControllableProcess.java b/geode-core/src/main/java/org/apache/geode/internal/process/ControllableProcess.java
index 288a4f3..7641ed3 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ControllableProcess.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ControllableProcess.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.process.ControlFileWatchdog.ControlRequestHandl
  * 
  * @since GemFire 8.0
  */
-public final class ControllableProcess {
+public class ControllableProcess {
   private static final Logger logger = LogService.getLogger();
 
   private final File workingDir;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/FileAlreadyExistsException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/FileAlreadyExistsException.java b/geode-core/src/main/java/org/apache/geode/internal/process/FileAlreadyExistsException.java
index 1bddfeb..4bb6d57 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/FileAlreadyExistsException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/FileAlreadyExistsException.java
@@ -20,7 +20,7 @@ package org.apache.geode.internal.process;
  * 
  * @since GemFire 7.0
  */
-public final class FileAlreadyExistsException extends Exception {
+public class FileAlreadyExistsException extends Exception {
   private static final long serialVersionUID = 5471082555536094256L;
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessController.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessController.java b/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessController.java
index efca868..fbea19e 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessController.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessController.java
@@ -47,7 +47,7 @@ import com.sun.tools.attach.VirtualMachine;
  * @since GemFire 7.0
  * @deprecated as of 8.0 please use {@link ControllableProcess} instead
  */
-public final class LocalProcessController {
+public class LocalProcessController {
 
   /** Property name for the JMX local connector address (from sun.management.Agent) */
   private static final String LOCAL_CONNECTOR_ADDRESS_PROP =

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessLauncher.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessLauncher.java b/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessLauncher.java
index 37184fc..04809c2 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessLauncher.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessLauncher.java
@@ -34,7 +34,7 @@ import java.io.IOException;
  * 
  * @since GemFire 7.0
  */
-public final class LocalProcessLauncher {
+public class LocalProcessLauncher {
 
   public static final String PROPERTY_IGNORE_IS_PID_ALIVE =
       DistributionConfig.GEMFIRE_PREFIX + "test.LocalProcessLauncher.ignoreIsPidAlive";

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/MBeanInvocationFailedException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/MBeanInvocationFailedException.java b/geode-core/src/main/java/org/apache/geode/internal/process/MBeanInvocationFailedException.java
index f1e499e..724a4d7 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/MBeanInvocationFailedException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/MBeanInvocationFailedException.java
@@ -19,7 +19,7 @@ package org.apache.geode.internal.process;
  * 
  * @since GemFire 7.0
  */
-public final class MBeanInvocationFailedException extends Exception {
+public class MBeanInvocationFailedException extends Exception {
   private static final long serialVersionUID = 7991096466859690801L;
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/MBeanProcessController.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/MBeanProcessController.java b/geode-core/src/main/java/org/apache/geode/internal/process/MBeanProcessController.java
index 0b7cfd6..ea5946e 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/MBeanProcessController.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/MBeanProcessController.java
@@ -42,7 +42,7 @@ import com.sun.tools.attach.VirtualMachine;
  * 
  * @since GemFire 8.0
  */
-public final class MBeanProcessController implements ProcessController {
+public class MBeanProcessController implements ProcessController {
 
   /** Property name for the JMX local connector address (from sun.management.Agent) */
   private static final String LOCAL_CONNECTOR_ADDRESS_PROP =

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/NativeProcessUtils.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/NativeProcessUtils.java b/geode-core/src/main/java/org/apache/geode/internal/process/NativeProcessUtils.java
index 685f39e..34cf81f 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/NativeProcessUtils.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/NativeProcessUtils.java
@@ -22,7 +22,7 @@ import org.apache.geode.internal.shared.NativeCalls;
  * 
  * @since GemFire 8.0
  */
-final class NativeProcessUtils implements InternalProcessUtils {
+class NativeProcessUtils implements InternalProcessUtils {
 
   private final static NativeCalls nativeCalls = NativeCalls.getInstance();
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/NonBlockingProcessStreamReader.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/NonBlockingProcessStreamReader.java b/geode-core/src/main/java/org/apache/geode/internal/process/NonBlockingProcessStreamReader.java
index 202790f..d5c1116 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/NonBlockingProcessStreamReader.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/NonBlockingProcessStreamReader.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.util.StopWatch;
  * 
  * @since GemFire 8.2
  */
-public final class NonBlockingProcessStreamReader extends ProcessStreamReader {
+public class NonBlockingProcessStreamReader extends ProcessStreamReader {
   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/process/PidUnavailableException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/PidUnavailableException.java b/geode-core/src/main/java/org/apache/geode/internal/process/PidUnavailableException.java
index d45ce59..6fa1c9f 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/PidUnavailableException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/PidUnavailableException.java
@@ -20,7 +20,7 @@ package org.apache.geode.internal.process;
  * 
  * @since GemFire 7.0
  */
-public final class PidUnavailableException extends Exception {
+public class PidUnavailableException extends Exception {
   private static final long serialVersionUID = -1660269538268828059L;
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ProcessControllerFactory.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessControllerFactory.java b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessControllerFactory.java
index 9c5c722..38fdcf8 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessControllerFactory.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessControllerFactory.java
@@ -27,7 +27,7 @@ import java.util.concurrent.TimeoutException;
  * 
  * @since GemFire 8.0
  */
-public final class ProcessControllerFactory {
+public class ProcessControllerFactory {
 
   public static final String PROPERTY_DISABLE_ATTACH_API =
       DistributionConfig.GEMFIRE_PREFIX + "test.ProcessControllerFactory.DisableAttachApi";

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ProcessLauncherContext.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessLauncherContext.java b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessLauncherContext.java
index 11a4709..463fd18 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessLauncherContext.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessLauncherContext.java
@@ -28,7 +28,7 @@ import java.util.Properties;
  * 
  * @since GemFire 7.0
  */
-public final class ProcessLauncherContext {
+public class ProcessLauncherContext {
 
   public static final String OVERRIDDEN_DEFAULTS_PREFIX =
       DistributionConfig.GEMFIRE_PREFIX + "default.";

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ProcessTerminatedAbnormallyException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessTerminatedAbnormallyException.java b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessTerminatedAbnormallyException.java
index 53c01f6..510353c 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessTerminatedAbnormallyException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessTerminatedAbnormallyException.java
@@ -26,7 +26,7 @@ import org.apache.geode.GemFireException;
  * @see org.apache.geode.GemFireException
  * @since GemFire 7.0
  */
-public final class ProcessTerminatedAbnormallyException extends GemFireException {
+public class ProcessTerminatedAbnormallyException extends GemFireException {
   private static final long serialVersionUID = -1181367425266595492L;
   private final int exitValue;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/ProcessUtils.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessUtils.java b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessUtils.java
index f15489f..3ff74d8 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/ProcessUtils.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/ProcessUtils.java
@@ -27,7 +27,7 @@ import org.apache.geode.internal.util.IOUtils;
  * 
  * @since GemFire 7.0
  */
-public final class ProcessUtils {
+public class ProcessUtils {
 
   private static InternalProcessUtils internal = initializeInternalProcessUtils();
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/process/UnableToControlProcessException.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/process/UnableToControlProcessException.java b/geode-core/src/main/java/org/apache/geode/internal/process/UnableToControlProcessException.java
index b89b886..ac4e4c8 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/process/UnableToControlProcessException.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/process/UnableToControlProcessException.java
@@ -20,7 +20,7 @@ package org.apache.geode.internal.process;
  * 
  * @since GemFire 8.0
  */
-public final class UnableToControlProcessException extends Exception {
+public class UnableToControlProcessException extends Exception {
   private static final long serialVersionUID = 7579463534993125290L;
 
   /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/shared/NativeCallsJNAImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/shared/NativeCallsJNAImpl.java b/geode-core/src/main/java/org/apache/geode/internal/shared/NativeCallsJNAImpl.java
index 92937c9..3efb62b 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/shared/NativeCallsJNAImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/shared/NativeCallsJNAImpl.java
@@ -59,7 +59,7 @@ import java.util.HashMap;
  * 
  * @since GemFire 8.0
  */
-public final class NativeCallsJNAImpl {
+public class NativeCallsJNAImpl {
 
   // no instance allowed
   private NativeCallsJNAImpl() {}
@@ -895,7 +895,7 @@ public final class NativeCallsJNAImpl {
   /**
    * Implementation of {@link NativeCalls} for Windows platforms.
    */
-  private static final class WinNativeCalls extends NativeCalls {
+  private static class WinNativeCalls extends NativeCalls {
 
     static {
       // for socket operations
@@ -903,7 +903,7 @@ public final class NativeCallsJNAImpl {
     }
 
     @SuppressWarnings("unused")
-    public static final class TcpKeepAlive extends Structure {
+    public static class TcpKeepAlive extends Structure {
       public int onoff;
       public int keepalivetime;
       public int keepaliveinterval;
@@ -921,7 +921,7 @@ public final class NativeCallsJNAImpl {
     static final int WSAENOPROTOOPT = 10042;
     static final int SIO_KEEPALIVE_VALS = -1744830460;
 
-    private static final class Kernel32 {
+    private static class Kernel32 {
 
       static {
         // kernel32 requires stdcall calling convention

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/statistics/CounterMonitor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/statistics/CounterMonitor.java b/geode-core/src/main/java/org/apache/geode/internal/statistics/CounterMonitor.java
index 7e0ce4f..d42ff4b 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/statistics/CounterMonitor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/statistics/CounterMonitor.java
@@ -17,7 +17,7 @@ package org.apache.geode.internal.statistics;
 /**
  * @since GemFire 7.0
  */
-public final class CounterMonitor extends StatisticsMonitor {
+public class CounterMonitor extends StatisticsMonitor {
 
   public static enum Type {
     GREATER_THAN, LESS_THAN

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/statistics/GaugeMonitor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/statistics/GaugeMonitor.java b/geode-core/src/main/java/org/apache/geode/internal/statistics/GaugeMonitor.java
index 3b07337..8990b6e 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/statistics/GaugeMonitor.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/statistics/GaugeMonitor.java
@@ -17,7 +17,7 @@ package org.apache.geode.internal.statistics;
 /**
  * @since GemFire 7.0
  */
-public final class GaugeMonitor extends StatisticsMonitor {
+public class GaugeMonitor extends StatisticsMonitor {
 
   private final Number lowThreshold;
   private final Number highThreshold;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/statistics/GemFireStatSampler.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/statistics/GemFireStatSampler.java b/geode-core/src/main/java/org/apache/geode/internal/statistics/GemFireStatSampler.java
index d4698ef..f264238 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/statistics/GemFireStatSampler.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/statistics/GemFireStatSampler.java
@@ -46,7 +46,7 @@ import org.apache.logging.log4j.Logger;
  * The StatisticsManager is implemented by DistributedSystem.
  * 
  */
-public final class GemFireStatSampler extends HostStatSampler {
+public class GemFireStatSampler extends HostStatSampler {
 
   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/statistics/SimpleStatisticId.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/statistics/SimpleStatisticId.java b/geode-core/src/main/java/org/apache/geode/internal/statistics/SimpleStatisticId.java
index 64fc2f0..57c1d0a 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/statistics/SimpleStatisticId.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/statistics/SimpleStatisticId.java
@@ -26,7 +26,7 @@ import org.apache.geode.Statistics;
  * 
  * @since GemFire 7.0
  */
-public final class SimpleStatisticId implements StatisticId {
+public class SimpleStatisticId implements StatisticId {
 
   private final StatisticDescriptor descriptor;
   private final Statistics statistics;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/statistics/StatisticDescriptorImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/statistics/StatisticDescriptorImpl.java b/geode-core/src/main/java/org/apache/geode/internal/statistics/StatisticDescriptorImpl.java
index b280c6d..476d118 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/statistics/StatisticDescriptorImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/statistics/StatisticDescriptorImpl.java
@@ -26,7 +26,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
  *
  * @since GemFire 3.0
  */
-public final class StatisticDescriptorImpl implements StatisticDescriptor {
+public class StatisticDescriptorImpl implements StatisticDescriptor {
 
   /** A constant for an <code>byte</code> type */
   static final byte BYTE = (byte) 3;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/tcp/MsgStreamerList.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/tcp/MsgStreamerList.java b/geode-core/src/main/java/org/apache/geode/internal/tcp/MsgStreamerList.java
index 87d9242..6797a11 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/tcp/MsgStreamerList.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/tcp/MsgStreamerList.java
@@ -30,7 +30,7 @@ import org.apache.geode.internal.logging.LogService;
  * 
  * @since GemFire 7.1
  */
-public final class MsgStreamerList implements BaseMsgStreamer {
+public class MsgStreamerList implements BaseMsgStreamer {
   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/tcp/VersionedMsgStreamer.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/tcp/VersionedMsgStreamer.java b/geode-core/src/main/java/org/apache/geode/internal/tcp/VersionedMsgStreamer.java
index 882cccf..de9c312 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/tcp/VersionedMsgStreamer.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/tcp/VersionedMsgStreamer.java
@@ -27,7 +27,7 @@ import org.apache.geode.internal.VersionedDataStream;
  * 
  * @since GemFire 7.1
  */
-final class VersionedMsgStreamer extends MsgStreamer implements VersionedDataStream {
+class VersionedMsgStreamer extends MsgStreamer implements VersionedDataStream {
 
   private final Version version;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/util/TransformUtils.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/util/TransformUtils.java b/geode-core/src/main/java/org/apache/geode/internal/util/TransformUtils.java
index 403c1d2..fa540fa 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/util/TransformUtils.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/util/TransformUtils.java
@@ -25,7 +25,7 @@ import org.apache.geode.internal.cache.persistence.PersistentMemberID;
 /**
  * Contains common data tranformation utility methods and transformers.
  */
-public final class TransformUtils {
+public class TransformUtils {
 
   /**
    * Transforms PersistentMemberIDs to a user friendly log entry.

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/internal/util/concurrent/CustomEntryConcurrentHashMap.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/util/concurrent/CustomEntryConcurrentHashMap.java b/geode-core/src/main/java/org/apache/geode/internal/util/concurrent/CustomEntryConcurrentHashMap.java
index 94e55b0..35c61c8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/util/concurrent/CustomEntryConcurrentHashMap.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/util/concurrent/CustomEntryConcurrentHashMap.java
@@ -305,7 +305,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
    * Segment.readValueUnderLock method is used as a backup in case a null (pre-initialized) value is
    * ever seen in an unsynchronized access method.
    */
-  static final class HashEntryImpl<K, V> implements HashEntry<K, V> {
+  static class HashEntryImpl<K, V> implements HashEntry<K, V> {
 
     protected final K key;
 
@@ -1069,7 +1069,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
    * 
    * @since GemFire 7.0
    */
-  static final class IdentitySegment<K, V> extends Segment<K, V> implements Serializable {
+  static class IdentitySegment<K, V> extends Segment<K, V> implements Serializable {
 
     private static final long serialVersionUID = 3086228147110819882L;
 
@@ -1212,7 +1212,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     }
   }
 
-  static final class DefaultHashEntryCreator<K, V> implements HashEntryCreator<K, V>, Serializable {
+  static class DefaultHashEntryCreator<K, V> implements HashEntryCreator<K, V>, Serializable {
 
     private static final long serialVersionUID = 3765680607280951726L;
 
@@ -2064,7 +2064,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     }
   }
 
-  final class KeyIterator extends HashIterator implements Iterator<K>, Enumeration<K> {
+  class KeyIterator extends HashIterator implements Iterator<K>, Enumeration<K> {
 
     public K next() {
       return super.nextEntry().getKey();
@@ -2075,7 +2075,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     }
   }
 
-  final class ValueIterator extends HashIterator implements Iterator<V>, Enumeration<V> {
+  class ValueIterator extends HashIterator implements Iterator<V>, Enumeration<V> {
 
     public V next() {
       return super.nextEntry().getMapValue();
@@ -2223,7 +2223,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
    * Custom Entry class used by EntryIterator.next(), that relays setValue changes to the underlying
    * map.
    */
-  final class WriteThroughEntry extends SimpleReusableEntry {
+  class WriteThroughEntry extends SimpleReusableEntry {
 
     private static final long serialVersionUID = -6364816773849437756L;
 
@@ -2254,7 +2254,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     }
   }
 
-  final class EntryIterator extends HashIterator implements Iterator<Map.Entry<K, V>> {
+  class EntryIterator extends HashIterator implements Iterator<Map.Entry<K, V>> {
 
     // GemStone change
     // added possibility to reuse a single Map.Entry for entire iteration
@@ -2276,7 +2276,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     // End GemStone change
   }
 
-  final class KeySet extends AbstractSet<K> {
+  class KeySet extends AbstractSet<K> {
 
     @Override
     public Iterator<K> iterator() {
@@ -2304,7 +2304,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     }
   }
 
-  final class Values extends AbstractCollection<V> {
+  class Values extends AbstractCollection<V> {
 
     @Override
     public Iterator<V> iterator() {
@@ -2327,7 +2327,7 @@ public class CustomEntryConcurrentHashMap<K, V> extends AbstractMap<K, V>
     }
   }
 
-  final class EntrySet extends AbstractSet<Map.Entry<K, V>> {
+  class EntrySet extends AbstractSet<Map.Entry<K, V>> {
 
     // GemStone change
     // added possibility to reuse a single Map.Entry for entire iteration

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/ArrayConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/ArrayConverter.java b/geode-core/src/main/java/org/apache/geode/management/internal/ArrayConverter.java
index cb203a0..1da9808 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/ArrayConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/ArrayConverter.java
@@ -27,7 +27,7 @@ import javax.management.openmbean.OpenDataException;
  * 
  * 
  */
-public final class ArrayConverter extends OpenTypeConverter {
+public class ArrayConverter extends OpenTypeConverter {
   ArrayConverter(Type targetType, ArrayType openArrayType, Class openArrayClass,
       OpenTypeConverter elementConverter) {
     super(targetType, openArrayType, openArrayClass);

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/CollectionConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/CollectionConverter.java b/geode-core/src/main/java/org/apache/geode/management/internal/CollectionConverter.java
index f469883..5d13516 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/CollectionConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/CollectionConverter.java
@@ -35,7 +35,7 @@ import javax.management.openmbean.OpenDataException;
  * 
  * 
  */
-public final class CollectionConverter extends OpenTypeConverter {
+public class CollectionConverter extends OpenTypeConverter {
   CollectionConverter(Type targetType, ArrayType openArrayType, Class openArrayClass,
       OpenTypeConverter elementConverter) {
     super(targetType, openArrayType, openArrayClass);

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/EnumConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/EnumConverter.java b/geode-core/src/main/java/org/apache/geode/management/internal/EnumConverter.java
index 320e66f..8923c08 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/EnumConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/EnumConverter.java
@@ -23,7 +23,7 @@ import javax.management.openmbean.SimpleType;
  * 
  * 
  */
-public final class EnumConverter<T extends Enum<T>> extends OpenTypeConverter {
+public class EnumConverter<T extends Enum<T>> extends OpenTypeConverter {
 
   EnumConverter(Class<T> enumClass) {
     super(enumClass, SimpleType.STRING, String.class);

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/IdentityConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/IdentityConverter.java b/geode-core/src/main/java/org/apache/geode/management/internal/IdentityConverter.java
index dcdb578..1d3a024 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/IdentityConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/IdentityConverter.java
@@ -24,7 +24,7 @@ import javax.management.openmbean.OpenType;
  * 
  *
  */
-public final class IdentityConverter extends OpenTypeConverter {
+public class IdentityConverter extends OpenTypeConverter {
   IdentityConverter(Type targetType, OpenType openType, Class openClass) {
     super(targetType, openType, openClass);
   }

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/OpenTypeConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/OpenTypeConverter.java b/geode-core/src/main/java/org/apache/geode/management/internal/OpenTypeConverter.java
index 6a0b842..f4af2de 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/OpenTypeConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/OpenTypeConverter.java
@@ -91,8 +91,7 @@ public abstract class OpenTypeConverter {
 
   private final Class openClass;
 
-  private static final class ConverterMap
-      extends WeakHashMap<Type, WeakReference<OpenTypeConverter>> {
+  private static class ConverterMap extends WeakHashMap<Type, WeakReference<OpenTypeConverter>> {
   }
 
   private static final ConverterMap converterMap = new ConverterMap();
@@ -513,7 +512,7 @@ public abstract class OpenTypeConverter {
    * 
    * 
    */
-  protected static final class CompositeBuilderViaFrom extends CompositeBuilder {
+  protected static class CompositeBuilderViaFrom extends CompositeBuilder {
 
     CompositeBuilderViaFrom(Class targetClass, String[] itemNames) {
       super(targetClass, itemNames);
@@ -669,7 +668,7 @@ public abstract class OpenTypeConverter {
    * 
    * 
    */
-  protected static final class CompositeBuilderViaConstructor extends CompositeBuilder {
+  protected static class CompositeBuilderViaConstructor extends CompositeBuilder {
 
     CompositeBuilderViaConstructor(Class targetClass, String[] itemNames) {
       super(targetClass, itemNames);
@@ -853,7 +852,7 @@ public abstract class OpenTypeConverter {
    * 
    * 
    */
-  protected static final class CompositeBuilderViaProxy extends CompositeBuilder {
+  protected static class CompositeBuilderViaProxy extends CompositeBuilder {
 
     CompositeBuilderViaProxy(Class targetClass, String[] itemNames) {
       super(targetClass, itemNames);

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/SystemManagementService.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/SystemManagementService.java b/geode-core/src/main/java/org/apache/geode/management/internal/SystemManagementService.java
index c674e48..fc8eb97 100755
--- a/geode-core/src/main/java/org/apache/geode/management/internal/SystemManagementService.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/SystemManagementService.java
@@ -59,7 +59,7 @@ import org.apache.geode.management.membership.MembershipListener;
  * 
  * @since GemFire 7.0
  */
-public final class SystemManagementService extends BaseManagementService {
+public class SystemManagementService extends BaseManagementService {
   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/management/internal/TableConverter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/TableConverter.java b/geode-core/src/main/java/org/apache/geode/management/internal/TableConverter.java
index b5e4ca3..5b18065 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/TableConverter.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/TableConverter.java
@@ -34,7 +34,7 @@ import javax.management.openmbean.TabularType;
  * 
  * 
  */
-public final class TableConverter extends OpenTypeConverter {
+public class TableConverter extends OpenTypeConverter {
   TableConverter(Type targetType, boolean sortedMap, TabularType tabularType,
       OpenTypeConverter keyConverter, OpenTypeConverter valueConverter) {
     super(targetType, tabularType, TabularData.class);

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/VMStatsMonitor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/VMStatsMonitor.java b/geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/VMStatsMonitor.java
index 6d22a8f..dd24942 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/VMStatsMonitor.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/VMStatsMonitor.java
@@ -28,7 +28,7 @@ import org.apache.geode.management.internal.MBeanJMXAdapter;
  * 
  * 
  */
-public final class VMStatsMonitor extends MBeanStatsMonitor {
+public class VMStatsMonitor extends MBeanStatsMonitor {
 
   private volatile float cpuUsage = 0;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/cli/Launcher.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/Launcher.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/Launcher.java
index c5c73dc..e4e00da 100755
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/Launcher.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/Launcher.java
@@ -73,7 +73,7 @@ import joptsimple.OptionSet;
  *
  * @since GemFire 7.0
  */
-public final class Launcher {
+public class Launcher {
   private static final String EXECUTE_OPTION = "execute";
   private static final String HELP_OPTION = "help";
   private static final String HELP = CliStrings.HELP;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java
index 0cf7078..9500677 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java
@@ -57,7 +57,7 @@ import java.util.Set;
  * @see org.apache.geode.management.internal.cli.commands.AbstractCommandsSupport
  * @since GemFire 7.0
  */
-public final class DeployCommands extends AbstractCommandsSupport {
+public class DeployCommands extends AbstractCommandsSupport {
 
   private final DeployFunction deployFunction = new DeployFunction();
   private final UndeployFunction undeployFunction = new UndeployFunction();

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java
index d61e72d..101bae4 100755
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java
@@ -2614,7 +2614,7 @@ public class LauncherLifecycleCommands extends AbstractCommandsSupport {
     return true;
   }
 
-  protected static final class LauncherSignalListener implements SignalListener {
+  protected static class LauncherSignalListener implements SignalListener {
 
     private volatile boolean signaled = false;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportedLogsSizeInfo.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportedLogsSizeInfo.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportedLogsSizeInfo.java
index 575930b..c175e1a 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportedLogsSizeInfo.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportedLogsSizeInfo.java
@@ -22,7 +22,7 @@ import java.io.DataOutput;
 import java.io.IOException;
 import java.util.Objects;
 
-public final class ExportedLogsSizeInfo implements DataSerializable {
+public class ExportedLogsSizeInfo implements DataSerializable {
   private long logsSize;
   private long diskAvailable;
   private long diskSize;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/cli/util/RegionAttributesDefault.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/util/RegionAttributesDefault.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/util/RegionAttributesDefault.java
index 049a3d6..1cbb9e2 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/util/RegionAttributesDefault.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/util/RegionAttributesDefault.java
@@ -30,7 +30,7 @@ import org.apache.geode.management.internal.cli.domain.PartitionAttributesInfo;
  * Contains the default values for the region attributes
  *
  */
-public final class RegionAttributesDefault {
+public class RegionAttributesDefault {
   public static final Scope SCOPE = AbstractRegion.DEFAULT_SCOPE.DISTRIBUTED_NO_ACK;
   public static final DataPolicy DATA_POLICY = DataPolicy.DEFAULT;
   public static final boolean CLONING_ENABLED = false;

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/configuration/domain/XmlEntity.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/configuration/domain/XmlEntity.java b/geode-core/src/main/java/org/apache/geode/management/internal/configuration/domain/XmlEntity.java
index f740dde..c630aa9 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/configuration/domain/XmlEntity.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/configuration/domain/XmlEntity.java
@@ -461,7 +461,7 @@ public class XmlEntity implements VersionedDataSerializable {
    *
    * @since GemFire 8.1
    */
-  public static final class XmlEntityBuilder {
+  public static class XmlEntityBuilder {
     private XmlEntity xmlEntity;
 
     /**

http://git-wip-us.apache.org/repos/asf/geode/blob/3b867342/geode-core/src/main/java/org/apache/geode/management/internal/configuration/utils/XmlConstants.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/configuration/utils/XmlConstants.java b/geode-core/src/main/java/org/apache/geode/management/internal/configuration/utils/XmlConstants.java
index 4a0579a..b00e287 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/configuration/utils/XmlConstants.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/configuration/utils/XmlConstants.java
@@ -21,7 +21,7 @@ import javax.xml.XMLConstants;
  *
  * @since GemFire 8.1
  */
-public final class XmlConstants {
+public class XmlConstants {
 
   /**
    * Standard prefix for {@link XMLConstants#W3C_XML_SCHEMA_INSTANCE_NS_URI}