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/16 19:29:03 UTC
[31/33] geode git commit: GEODE-2929: remove superfluous uses of
final from internal classes
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);