geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From u..@apache.org
Subject [31/69] [abbrv] geode git commit: GEODE-2929: remove superfluous uses of final from internal classes
Date Mon, 22 May 2017 18:29:27 GMT
http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/ObjIdConcurrentMap.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/ObjIdConcurrentMap.java b/geode-core/src/main/java/org/apache/geode/internal/ObjIdConcurrentMap.java
index b4f882e..17894ad 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/ObjIdConcurrentMap.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/ObjIdConcurrentMap.java
@@ -186,7 +186,7 @@ public class ObjIdConcurrentMap<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 HashEntry<V> {
+  static class HashEntry<V> {
     final int key;
     final int hash;
     volatile V value;
@@ -200,7 +200,7 @@ public class ObjIdConcurrentMap<V> /* extends AbstractMap<K, V> */
     }
 
     @SuppressWarnings("unchecked")
-    static final <V> HashEntry<V>[] newArray(int i) {
+    static <V> HashEntry<V>[] newArray(int i) {
       return new HashEntry[i];
     }
   }
@@ -209,7 +209,7 @@ public class ObjIdConcurrentMap<V> /* extends AbstractMap<K, V> */
    * Segments are specialized versions of hash tables. This subclasses from ReentrantLock
    * opportunistically, just to simplify some locking and avoid separate construction.
    */
-  static final class Segment<V> extends ReentrantLock implements Serializable {
+  static class Segment<V> extends ReentrantLock implements Serializable {
     /*
      * Segments maintain a table of entry lists that are ALWAYS kept in a consistent state, so can
      * be read without locking. Next fields of nodes are immutable (final). All list additions are
@@ -279,7 +279,7 @@ public class ObjIdConcurrentMap<V> /* extends AbstractMap<K, V> */
     }
 
     @SuppressWarnings("unchecked")
-    static final <K, V> Segment<V>[] newArray(int i) {
+    static <K, V> Segment<V>[] newArray(int i) {
       return new Segment[i];
     }
 
@@ -1049,12 +1049,11 @@ public class ObjIdConcurrentMap<V> /* extends AbstractMap<K, V> */
   }
 
   /*
-   * final class KeyIterator extends HashIterator implements Iterator<K>, Enumeration<K> { public K
-   * next() { return super.nextEntry().key; } public K nextElement() { return super.nextEntry().key;
-   * } }
+   * class KeyIterator extends HashIterator implements Iterator<K>, Enumeration<K> { public K next()
+   * { return super.nextEntry().key; } public K nextElement() { return super.nextEntry().key; } }
    * 
-   * final class ValueIterator extends HashIterator implements Iterator<V>, Enumeration<V> { public
-   * V next() { return super.nextEntry().value; } public V nextElement() { return
+   * class ValueIterator extends HashIterator implements Iterator<V>, Enumeration<V> { public V
+   * next() { return super.nextEntry().value; } public V nextElement() { return
    * super.nextEntry().value; } }
    */
 
@@ -1122,7 +1121,7 @@ public class ObjIdConcurrentMap<V> /* extends AbstractMap<K, V> */
   // * Custom Entry class used by EntryIterator.next(), that relays
   // * setValue changes to the underlying map.
   // */
-  // final class WriteThroughEntry
+  // class WriteThroughEntry
   // extends SimpleEntry<V>
   // {
   // WriteThroughEntry(int k, V v) {
@@ -1148,25 +1147,25 @@ public class ObjIdConcurrentMap<V> /* extends AbstractMap<K, V> */
   // }
 
   /*
-   * final class EntryIterator extends HashIterator implements Iterator<Entry<V>> { public Entry<V>
-   * next() { HashEntry<V> e = super.nextEntry(); return new WriteThroughEntry(e.key, e.value); } }
+   * class EntryIterator extends HashIterator implements Iterator<Entry<V>> { public Entry<V> next()
+   * { HashEntry<V> e = super.nextEntry(); return new WriteThroughEntry(e.key, e.value); } }
    * 
-   * final class KeySet extends AbstractSet<K> { public Iterator<K> iterator() { return new
-   * KeyIterator(); } public int size() { return ObjIdConcurrentMap.this.size(); } public boolean
-   * isEmpty() { return ObjIdConcurrentMap.this.isEmpty(); } public boolean contains(Object o) {
-   * return ObjIdConcurrentMap.this.containsKey(o); } public boolean remove(Object o) { return
+   * class KeySet extends AbstractSet<K> { public Iterator<K> iterator() { return new KeyIterator();
+   * } public int size() { return ObjIdConcurrentMap.this.size(); } public boolean isEmpty() {
+   * return ObjIdConcurrentMap.this.isEmpty(); } public boolean contains(Object o) { return
+   * ObjIdConcurrentMap.this.containsKey(o); } public boolean remove(Object o) { return
    * ObjIdConcurrentMap.this.remove(o) != null; } public void clear() {
    * ObjIdConcurrentMap.this.clear(); } }
    * 
-   * final class Values extends AbstractCollection<V> { public Iterator<V> iterator() { return new
+   * class Values extends AbstractCollection<V> { public Iterator<V> iterator() { return new
    * ValueIterator(); } public int size() { return ObjIdConcurrentMap.this.size(); } public boolean
    * isEmpty() { return ObjIdConcurrentMap.this.isEmpty(); } public boolean contains(Object o) {
    * return ObjIdConcurrentMap.this.containsValue(o); } public void clear() {
    * ObjIdConcurrentMap.this.clear(); } }
    * 
-   * final class EntrySet extends AbstractSet<Map.Entry<K,V>> { public Iterator<Map.Entry<K,V>>
-   * iterator() { return new EntryIterator(); } public boolean contains(Object o) { if (!(o
-   * instanceof Map.Entry)) return false; Map.Entry<?,?> e = (Map.Entry<?,?>)o; V v =
+   * class EntrySet extends AbstractSet<Map.Entry<K,V>> { public Iterator<Map.Entry<K,V>> iterator()
+   * { return new EntryIterator(); } public boolean contains(Object o) { if (!(o instanceof
+   * Map.Entry)) return false; Map.Entry<?,?> e = (Map.Entry<?,?>)o; V v =
    * ObjIdConcurrentMap.this.get(e.getKey()); return v != null && v.equals(e.getValue()); } public
    * boolean remove(Object o) { if (!(o instanceof Map.Entry)) return false; Map.Entry<?,?> e =
    * (Map.Entry<?,?>)o; return ObjIdConcurrentMap.this.remove(e.getKey(), e.getValue()); } public

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/PureJavaMode.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/PureJavaMode.java b/geode-core/src/main/java/org/apache/geode/internal/PureJavaMode.java
index bc6f23e..4e60c0b 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/PureJavaMode.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/PureJavaMode.java
@@ -20,7 +20,7 @@ import org.apache.geode.distributed.internal.DistributionConfig;
 /**
  * Used to determine if product should use pure java mode.
  */
-public final class PureJavaMode {
+public class PureJavaMode {
   /**
    * System property to set to true to force pure java mode
    */
@@ -67,11 +67,11 @@ public final class PureJavaMode {
     osStatsAreAvailable = osName.startsWith("Linux") || !isPure;
   }
 
-  public final static boolean isPure() {
+  public static boolean isPure() {
     return isPure;
   }
 
-  public final static boolean is64Bit() {
+  public static boolean is64Bit() {
     return is64Bit;
   }
 
@@ -79,7 +79,7 @@ public final class PureJavaMode {
    * Linux has OsStats even in PureJava mode but other platforms require the native code to provide
    * OS Statistics. return true if OSStatistics are available
    */
-  public final static boolean osStatsAreAvailable() {
+  public static boolean osStatsAreAvailable() {
     return osStatsAreAvailable;
   }
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/SetUtils.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/SetUtils.java b/geode-core/src/main/java/org/apache/geode/internal/SetUtils.java
index 506cf03..66bd337 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/SetUtils.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/SetUtils.java
@@ -24,7 +24,7 @@ import java.util.Set;
  * @since GemFire 59poc
  * 
  */
-public final class SetUtils {
+public class SetUtils {
   public static <T> boolean intersectsWith(final Set<? extends T> a, final Set<? extends T> b) {
     if (a == b) {
       return true;

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/SystemAdmin.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/SystemAdmin.java b/geode-core/src/main/java/org/apache/geode/internal/SystemAdmin.java
index bdb635e..ceae486 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/SystemAdmin.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/SystemAdmin.java
@@ -1267,7 +1267,7 @@ public class SystemAdmin {
    *
    */
   @SuppressWarnings("serial")
-  private static final class CombinedResources extends ArrayList<ResourceInst> {
+  private static class CombinedResources extends ArrayList<ResourceInst> {
     public CombinedResources(StatValue v) {
       super(Arrays.asList(v.getResources()));
     }

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/Version.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/Version.java b/geode-core/src/main/java/org/apache/geode/internal/Version.java
index 288d104..e058edb 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/Version.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/Version.java
@@ -34,7 +34,7 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
  * 
  * @since GemFire 5.7
  */
-public final class Version implements Comparable<Version> {
+public class Version implements Comparable<Version> {
 
   /** The name of this version */
   private final transient String name;
@@ -318,7 +318,7 @@ public final class Version implements Comparable<Version> {
    *        {@link #readOrdinal(DataInput)} can deal with both compressed/uncompressed cases
    *        seemlessly
    */
-  public final void writeOrdinal(DataOutput out, boolean compressed) throws IOException {
+  public void writeOrdinal(DataOutput out, boolean compressed) throws IOException {
     writeOrdinal(out, this.ordinal, compressed);
   }
 
@@ -326,7 +326,7 @@ public final class Version implements Comparable<Version> {
    * Fixed number of bytes required for serializing this version when "compressed" flag is false in
    * {@link #writeOrdinal(DataOutput, boolean)}.
    */
-  public static final int uncompressedSize() {
+  public static int uncompressedSize() {
     return 3;
   }
 
@@ -334,7 +334,7 @@ public final class Version implements Comparable<Version> {
    * Fixed number of bytes required for serializing this version when "compressed" flag is true in
    * {@link #writeOrdinal(DataOutput, boolean)}.
    */
-  public final int compressedSize() {
+  public int compressedSize() {
     if (ordinal <= Byte.MAX_VALUE) {
       return 1;
     } else {
@@ -438,35 +438,35 @@ public final class Version implements Comparable<Version> {
     return this.gemfireVersion != null ? this.gemfireVersion : this;
   }
 
-  public final String getMethodSuffix() {
+  public String getMethodSuffix() {
     return this.methodSuffix;
   }
 
-  public final String getProductName() {
+  public String getProductName() {
     return this.productName;
   }
 
-  public final String getName() {
+  public String getName() {
     return this.name;
   }
 
-  public final short getMajorVersion() {
+  public short getMajorVersion() {
     return this.majorVersion;
   }
 
-  public final short getMinorVersion() {
+  public short getMinorVersion() {
     return this.minorVersion;
   }
 
-  public final short getRelease() {
+  public short getRelease() {
     return this.release;
   }
 
-  public final short getPatch() {
+  public short getPatch() {
     return this.patch;
   }
 
-  public final short ordinal() {
+  public short ordinal() {
     return this.ordinal;
   }
 
@@ -488,7 +488,7 @@ public final class Version implements Comparable<Version> {
    * @return negative if this version is older, positive if this version is newer, 0 if this is the
    *         same version
    */
-  public final int compareTo(short other) {
+  public int compareTo(short other) {
     // first try to find the actual Version object
     Version v = fromOrdinalNoThrow(other, false);
     if (v == null) {
@@ -502,7 +502,7 @@ public final class Version implements Comparable<Version> {
    * {@inheritDoc}
    */
   @Override
-  public final int compareTo(Version other) {
+  public int compareTo(Version other) {
     if (other != null) {
       // [bruce] old implementation used ordinals for comparison, but this requires
       // ordinals to be in increasing order, which may not always be possible

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/VersionedDataInputStream.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/VersionedDataInputStream.java b/geode-core/src/main/java/org/apache/geode/internal/VersionedDataInputStream.java
index a8d4b88..93511b4 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/VersionedDataInputStream.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/VersionedDataInputStream.java
@@ -25,7 +25,7 @@ import java.io.InputStream;
  * 
  * @since GemFire 7.1
  */
-public final class VersionedDataInputStream extends DataInputStream implements VersionedDataStream {
+public class VersionedDataInputStream extends DataInputStream implements VersionedDataStream {
 
   private final Version version;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/VersionedDataOutputStream.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/VersionedDataOutputStream.java b/geode-core/src/main/java/org/apache/geode/internal/VersionedDataOutputStream.java
index 2fb40b9..001fefe 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/VersionedDataOutputStream.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/VersionedDataOutputStream.java
@@ -23,8 +23,7 @@ import java.io.OutputStream;
  * 
  * @since GemFire 7.1
  */
-public final class VersionedDataOutputStream extends DataOutputStream
-    implements VersionedDataStream {
+public class VersionedDataOutputStream extends DataOutputStream implements VersionedDataStream {
 
   private final Version version;
 
@@ -43,7 +42,7 @@ public final class VersionedDataOutputStream extends DataOutputStream
    * {@inheritDoc}
    */
   @Override
-  public final Version getVersion() {
+  public Version getVersion() {
     return this.version;
   }
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectInput.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectInput.java b/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectInput.java
index 86e81c4..bd0b27f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectInput.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectInput.java
@@ -24,7 +24,7 @@ import java.io.ObjectInput;
  * 
  * @since GemFire 7.1
  */
-public final class VersionedObjectInput implements ObjectInput, VersionedDataStream {
+public class VersionedObjectInput implements ObjectInput, VersionedDataStream {
 
   private final ObjectInput in;
   private final Version version;

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectOutput.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectOutput.java b/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectOutput.java
index 83e6f73..09f3995 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectOutput.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/VersionedObjectOutput.java
@@ -24,7 +24,7 @@ import java.io.ObjectOutput;
  * 
  * @since GemFire 7.1
  */
-public final class VersionedObjectOutput implements ObjectOutput, VersionedDataStream {
+public class VersionedObjectOutput implements ObjectOutput, VersionedDataStream {
 
   private final ObjectOutput out;
   private final Version version;

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerRequest.java
index 5993d6b..6dd97e8 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerRequest.java
@@ -30,7 +30,7 @@ import java.io.*;
  * 
  * @since GemFire 3.5
  */
-public final class AddHealthListenerRequest extends AdminRequest {
+public class AddHealthListenerRequest extends AdminRequest {
   // instance variables
   private GemFireHealthConfig cfg;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerResponse.java
index 8b960ab..8777303 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddHealthListenerResponse.java
@@ -30,7 +30,7 @@ import org.apache.geode.distributed.internal.membership.*;
  * 
  * @since GemFire 3.5
  */
-public final class AddHealthListenerResponse extends AdminResponse {
+public class AddHealthListenerResponse extends AdminResponse {
   // instance variables
   int listenerId;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerRequest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerRequest.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerRequest.java
index 703ae6a..a911263 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerRequest.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerRequest.java
@@ -28,7 +28,7 @@ import java.io.*;
 /**
  * A message that is sent to a particular distribution manager to add a statistic listener.
  */
-public final class AddStatListenerRequest extends AdminRequest {
+public class AddStatListenerRequest extends AdminRequest {
   // instance variables
   private long resourceId;
   private String statName;

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerResponse.java
index b88772f..f5d708e 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AddStatListenerResponse.java
@@ -28,7 +28,7 @@ import org.apache.geode.internal.statistics.GemFireStatSampler;
  * A message that is sent to a particular distribution manager to get its current
  * <code>RemoteAddStatListener</code>.
  */
-public final class AddStatListenerResponse extends AdminResponse {
+public class AddStatListenerResponse extends AdminResponse {
   // instance variables
   int listenerId;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleDisconnectMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleDisconnectMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleDisconnectMessage.java
index 8452929..dd9b793 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleDisconnectMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleDisconnectMessage.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 disconnected.
  */
-public final class AdminConsoleDisconnectMessage extends PooledDistributionMessage {
+public class AdminConsoleDisconnectMessage extends PooledDistributionMessage {
   private static final Logger logger = LogService.getLogger();
 
   // instance variables

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleMessage.java
index fa07560..d8c9665 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminConsoleMessage.java
@@ -34,7 +34,7 @@ import org.apache.geode.internal.logging.log4j.LogWriterLogger;
  * 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 AdminConsoleMessage extends PooledDistributionMessage {
+public class AdminConsoleMessage extends PooledDistributionMessage {
   // instance variables
   int level;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminFailureResponse.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminFailureResponse.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminFailureResponse.java
index 002d7d7..d201575 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminFailureResponse.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AdminFailureResponse.java
@@ -26,7 +26,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * A response to a failed request.
  */
-public final class AdminFailureResponse extends AdminResponse {
+public class AdminFailureResponse extends AdminResponse {
   // instance variables
   Exception cause;
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertLevelChangeMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertLevelChangeMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertLevelChangeMessage.java
index c3a293f..25f1eb1 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertLevelChangeMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertLevelChangeMessage.java
@@ -37,7 +37,7 @@ import org.apache.geode.internal.logging.log4j.LogMarker;
  *
  * @since GemFire 3.5
  */
-public final class AlertLevelChangeMessage extends SerialDistributionMessage {
+public class AlertLevelChangeMessage extends SerialDistributionMessage {
 
   private static final Logger logger = LogService.getLogger();
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertListenerMessage.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertListenerMessage.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertListenerMessage.java
index 4f0e9ce..9f0e4bf 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertListenerMessage.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/AlertListenerMessage.java
@@ -30,8 +30,7 @@ import org.apache.geode.distributed.internal.membership.*;
 /**
  * A message that is sent to a particular console distribution manager to notify it of an alert.
  */
-public final class AlertListenerMessage extends PooledDistributionMessage
-    implements AdminMessageType {
+public class AlertListenerMessage extends PooledDistributionMessage implements AdminMessageType {
   // instance variables
   private int msgLevel;
   private Date msgDate;

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteRegionAttributes.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteRegionAttributes.java b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteRegionAttributes.java
index 7a6ad09..fdcb224 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteRegionAttributes.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteRegionAttributes.java
@@ -498,11 +498,11 @@ public class RemoteRegionAttributes implements RegionAttributes, DataSerializabl
     }
 
     @Override
-    public final String toString() {
+    public String toString() {
       return desc;
     }
 
-    public final void close() {}
+    public void close() {}
   }
 
   private static class RemoteCacheListener extends RemoteCacheCallback implements CacheListener {

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/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/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/BaseDecoratorImpl.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/BaseDecoratorImpl.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/BaseDecoratorImpl.java
index 0f752b3..5b7c750 100755
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/BaseDecoratorImpl.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/BaseDecoratorImpl.java
@@ -48,7 +48,7 @@ public abstract class BaseDecoratorImpl implements StatAlertDefinition {
     this._def = definition;
   }
 
-  public final int getId() {
+  public int getId() {
     return _def.getId();
   }
 
@@ -74,11 +74,11 @@ public abstract class BaseDecoratorImpl implements StatAlertDefinition {
     _def.setName(name);
   }
 
-  public final StatisticInfo[] getStatisticInfo() {
+  public StatisticInfo[] getStatisticInfo() {
     return _def.getStatisticInfo();
   }
 
-  public final void setStatisticInfo(StatisticInfo[] info) {
+  public void setStatisticInfo(StatisticInfo[] info) {
     _def.setStatisticInfo(info);
   }
 

http://git-wip-us.apache.org/repos/asf/geode/blob/a4458531/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/a4458531/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionHelper.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionHelper.java b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionHelper.java
index fa96cae..b1a8b0d 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionHelper.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/admin/statalerts/FunctionHelper.java
@@ -132,7 +132,7 @@ public class FunctionHelper {
    * 
    * @param vals Array of number
    */
-  public static final Number SUM(Number[] vals) {
+  public static Number SUM(Number[] vals) {
     try {
       double sum = 0.0;
       for (int i = 0; i < vals.length; i++) {
@@ -160,7 +160,7 @@ public class FunctionHelper {
    * 
    * @param vals Array of number
    */
-  public static final Number AVG(Number[] vals) {
+  public static Number AVG(Number[] vals) {
     try {
       return Double.valueOf(SUM(vals).doubleValue() / vals.length);
     } catch (VirtualMachineError err) {
@@ -184,7 +184,7 @@ public class FunctionHelper {
    * 
    * @param vals Array of number
    */
-  public static final Number MIN(Number[] vals) {
+  public static Number MIN(Number[] vals) {
     try {
       Collection col = Arrays.asList(vals);
       Number min = (Number) Collections.max(col);
@@ -211,7 +211,7 @@ public class FunctionHelper {
    * 
    * @param vals Array of number
    */
-  public static final Number MAX(Number[] vals) {
+  public static Number MAX(Number[] vals) {
     try {
       Collection col = Arrays.asList(vals);
       Number max = (Number) Collections.max(col);


Mime
View raw message