geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kl...@apache.org
Subject [02/18] geode git commit: GEODE-2980: remove unnecessary exlicit keywords from interfaces
Date Tue, 13 Jun 2017 17:17:24 GMT
http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/cli/ConverterHint.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/cli/ConverterHint.java b/geode-core/src/main/java/org/apache/geode/management/cli/ConverterHint.java
index 9c32559..4e1a1b2 100644
--- a/geode-core/src/main/java/org/apache/geode/management/cli/ConverterHint.java
+++ b/geode-core/src/main/java/org/apache/geode/management/cli/ConverterHint.java
@@ -23,26 +23,25 @@ import org.springframework.shell.core.annotation.CliOption;
  * @since GemFire 8.0
  */
 public interface ConverterHint {
-  public static final String DISABLE_STRING_CONVERTER = ":disable-string-converter";
-  public static final String DISKSTORE =
-      "geode.converter.cluster.diskstore" + DISABLE_STRING_CONVERTER;
-  public static final String FILE = "geode.converter.file";
-  public static final String FILE_PATH = "geode.converter.file.path" + DISABLE_STRING_CONVERTER;
-  public static final String HINT = "geode.converter.hint" + DISABLE_STRING_CONVERTER;
-  public static final String HELP = "geode.converter.help" + DISABLE_STRING_CONVERTER;
-  public static final String MEMBERGROUP = "geode.converter.member.groups";
+  String DISABLE_STRING_CONVERTER = ":disable-string-converter";
+  String DISKSTORE = "geode.converter.cluster.diskstore" + DISABLE_STRING_CONVERTER;
+  String FILE = "geode.converter.file";
+  String FILE_PATH = "geode.converter.file.path" + DISABLE_STRING_CONVERTER;
+  String HINT = "geode.converter.hint" + DISABLE_STRING_CONVERTER;
+  String HELP = "geode.converter.help" + DISABLE_STRING_CONVERTER;
+  String MEMBERGROUP = "geode.converter.member.groups";
   /** Hint to be used for all types of GemFire cluster members */
-  public static final String ALL_MEMBER_IDNAME = "geode.converter.all.member.idOrName";
+  String ALL_MEMBER_IDNAME = "geode.converter.all.member.idOrName";
   /** Hint to be used for all non locator GemFire cluster members */
-  public static final String MEMBERIDNAME = "geode.converter.member.idOrName";
+  String MEMBERIDNAME = "geode.converter.member.idOrName";
   /** Hint to be used for GemFire stand-alone locator members */
-  public static final String LOCATOR_MEMBER_IDNAME = "geode.converter.locatormember.idOrName";
+  String LOCATOR_MEMBER_IDNAME = "geode.converter.locatormember.idOrName";
   /** Hint to be used for configured locators for discovery */
-  public static final String LOCATOR_DISCOVERY_CONFIG = "geode.converter.locators.discovery.config";
-  public static final String REGION_PATH = "geode.converter.region.path" + DISABLE_STRING_CONVERTER;
-  public static final String INDEX_TYPE = "geode.converter.index.type";
-  public static final String GATEWAY_SENDER_ID = "geode.converter.gateway.senderid";
-  public static final String GATEWAY_RECEIVER_ID = "geode.converter.gateway.receiverid";
-  public static final String LOG_LEVEL = "geode.converter.log.levels" + DISABLE_STRING_CONVERTER;
+  String LOCATOR_DISCOVERY_CONFIG = "geode.converter.locators.discovery.config";
+  String REGION_PATH = "geode.converter.region.path" + DISABLE_STRING_CONVERTER;
+  String INDEX_TYPE = "geode.converter.index.type";
+  String GATEWAY_SENDER_ID = "geode.converter.gateway.senderid";
+  String GATEWAY_RECEIVER_ID = "geode.converter.gateway.receiverid";
+  String LOG_LEVEL = "geode.converter.log.levels" + DISABLE_STRING_CONVERTER;
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/cli/Result.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/cli/Result.java b/geode-core/src/main/java/org/apache/geode/management/cli/Result.java
index 57f7c43..62b8c49 100755
--- a/geode-core/src/main/java/org/apache/geode/management/cli/Result.java
+++ b/geode-core/src/main/java/org/apache/geode/management/cli/Result.java
@@ -36,7 +36,7 @@ public interface Result {
    * 
    * @since GemFire 7.0
    */
-  public enum Status {
+  enum Status {
     /**
      * Indicates that the command completed successfully.
      */
@@ -64,19 +64,19 @@ public interface Result {
   /**
    * Returns the status of a processed command.
    */
-  public Status getStatus();
+  Status getStatus();
 
   /**
    * Resets the pointer to the first line in the Result.
    */
-  public void resetToFirstLine();
+  void resetToFirstLine();
 
   /**
    * Returns whether the result has any more lines of information.
    * 
    * @return True if there are more lines, false otherwise.
    */
-  public boolean hasNextLine();
+  boolean hasNextLine();
 
   /**
    * Returns the next line of information from the Result.
@@ -84,14 +84,14 @@ public interface Result {
    * @throws IndexOutOfBoundsException if this method is called more times than there are lines of
    *         information.
    */
-  public String nextLine();
+  String nextLine();
 
   /**
    * Returns whether this Result has a file as a part of the command output.
    * 
    * @return True if there is a file, false otherwise.
    */
-  public boolean hasIncomingFiles();
+  boolean hasIncomingFiles();
 
   /**
    * Save the file(s) from this Result. {@link #hasIncomingFiles()} should be used before calling
@@ -101,7 +101,7 @@ public interface Result {
    * @throws IOException If an error occurs while saving the file.
    * @throws RuntimeException If there is no file in the Result to save.
    */
-  public void saveIncomingFiles(String directory) throws IOException;
+  void saveIncomingFiles(String directory) throws IOException;
 
   /****
    * Return whether the configuration changes due to command have been persisted to cluster
@@ -109,7 +109,7 @@ public interface Result {
    * 
    * @return True if the command has failed to persist configuration changes , false otherwise.
    */
-  public boolean failedToPersist();
+  boolean failedToPersist();
 
   /*****
    * Sets whether the command changes have not been persisted to the cluster configuration
@@ -117,6 +117,6 @@ public interface Result {
    * @param commandPersisted true if the command changes are persisted to the cluster configuration,
    *        false otherwise.
    */
-  public void setCommandPersisted(boolean commandPersisted);
+  void setCommandPersisted(boolean commandPersisted);
 }
 

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/ManagementConstants.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/ManagementConstants.java b/geode-core/src/main/java/org/apache/geode/management/internal/ManagementConstants.java
index 151aa9d..db0cb89 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/ManagementConstants.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/ManagementConstants.java
@@ -28,23 +28,23 @@ import java.util.concurrent.TimeUnit;
 public interface ManagementConstants {
 
   /* *********** Constant Strings used in Federation BEGIN ****************** */
-  public static final String MGMT_FUNCTION_ID = ManagementFunction.class.getName();
+  String MGMT_FUNCTION_ID = ManagementFunction.class.getName();
 
-  public static final String QUERY_DATA_FUNCTION = QueryDataFunction.class.getName();
+  String QUERY_DATA_FUNCTION = QueryDataFunction.class.getName();
 
-  public static final int DEFAULT_QUERY_LIMIT = 1000;
+  int DEFAULT_QUERY_LIMIT = 1000;
 
-  public static final long GII_WAIT_TIME_IN_MILLIS = 500;
+  long GII_WAIT_TIME_IN_MILLIS = 500;
 
-  public static final int NUM_THREADS_FOR_GII = 10;
+  int NUM_THREADS_FOR_GII = 10;
 
-  public static final int REFRESH_TIME = DistributionConfig.DEFAULT_JMX_MANAGER_UPDATE_RATE;
+  int REFRESH_TIME = DistributionConfig.DEFAULT_JMX_MANAGER_UPDATE_RATE;
 
-  public static final String MONITORING_REGION = "_monitoringRegion";
+  String MONITORING_REGION = "_monitoringRegion";
 
-  public static final String NOTIFICATION_REGION = "_notificationRegion";
+  String NOTIFICATION_REGION = "_notificationRegion";
 
-  public static final String cascadingSeparator = "/";
+  String cascadingSeparator = "/";
 
   /* *********** Constant Strings used in Federation END ******************** */
 
@@ -74,132 +74,126 @@ public interface ManagementConstants {
 
   /* ************ ObjectName Strings for MBeans **************************** */
   // 1. Basic elements
-  public static final String OBJECTNAME__DEFAULTDOMAIN = "GemFire";
+  String OBJECTNAME__DEFAULTDOMAIN = "GemFire";
 
   /**
    * Key value separator for ObjectName
    */
-  public static final String KEYVAL_SEPARATOR = ",";
+  String KEYVAL_SEPARATOR = ",";
 
   /**
    * Key value separator for ObjectName
    */
-  public static final String DOMAIN_SEPARATOR = ":";
+  String DOMAIN_SEPARATOR = ":";
 
   /**
    * Prefix used for all the ObjectName Strings
    */
-  public static final String OBJECTNAME__PREFIX = OBJECTNAME__DEFAULTDOMAIN + DOMAIN_SEPARATOR;
+  String OBJECTNAME__PREFIX = OBJECTNAME__DEFAULTDOMAIN + DOMAIN_SEPARATOR;
 
   // 2. Actual ObjectNames and/or ObjectName structures
-  public static final String OBJECTNAME__DISTRIBUTEDSYSTEM_MXBEAN =
+  String OBJECTNAME__DISTRIBUTEDSYSTEM_MXBEAN =
       OBJECTNAME__PREFIX + "service=System,type=Distributed";
 
-  public static final String OBJECTNAME__MEMBER_MXBEAN =
-      OBJECTNAME__PREFIX + "type=Member,member={0}";
+  String OBJECTNAME__MEMBER_MXBEAN = OBJECTNAME__PREFIX + "type=Member,member={0}";
 
-  public static final String OBJECTNAME__MANAGER_MXBEAN =
-      OBJECTNAME__PREFIX + "service=Manager,type=Member,member={0}";
+  String OBJECTNAME__MANAGER_MXBEAN = OBJECTNAME__PREFIX + "service=Manager,type=Member,member={0}";
 
-  public static final String OBJECTNAME__DISTRIBUTEDREGION_MXBEAN =
+  String OBJECTNAME__DISTRIBUTEDREGION_MXBEAN =
       OBJECTNAME__PREFIX + "service=Region,name={0},type=Distributed";
 
-  public static final String OBJECTNAME__REGION_MXBEAN =
+  String OBJECTNAME__REGION_MXBEAN =
       OBJECTNAME__PREFIX + "service=Region,name={0},type=Member,member={1}";
 
-  public static final String OBJECTNAME__DISTRIBUTEDLOCKSERVICE_MXBEAN =
+  String OBJECTNAME__DISTRIBUTEDLOCKSERVICE_MXBEAN =
       OBJECTNAME__PREFIX + "service=LockService,name={0},type=Distributed";
 
-  public static final String OBJECTNAME__LOCKSERVICE_MXBEAN =
+  String OBJECTNAME__LOCKSERVICE_MXBEAN =
       OBJECTNAME__PREFIX + "service=LockService,name={0},type=Member,member={1}";
 
-  public static final String OBJECTNAME__ASYNCEVENTQUEUE_MXBEAN =
+  String OBJECTNAME__ASYNCEVENTQUEUE_MXBEAN =
       OBJECTNAME__PREFIX + "service=AsyncEventQueue,queue={0},type=Member,member={1}";
 
-  public static final String OBJECTNAME__GATEWAYSENDER_MXBEAN =
+  String OBJECTNAME__GATEWAYSENDER_MXBEAN =
       OBJECTNAME__PREFIX + "service=GatewaySender,gatewaySender={0},type=Member,member={1}";
 
-  public static final String OBJECTNAME__GATEWAYRECEIVER_MXBEAN =
+  String OBJECTNAME__GATEWAYRECEIVER_MXBEAN =
       OBJECTNAME__PREFIX + "service=GatewayReceiver,type=Member,member={0}";
 
-  public static final String OBJECTNAME__CLIENTSERVICE_MXBEAN =
+  String OBJECTNAME__CLIENTSERVICE_MXBEAN =
       OBJECTNAME__PREFIX + "service=CacheServer,port={0},type=Member,member={1}";
 
-  public static final String OBJECTNAME__DISKSTORE_MXBEAN =
+  String OBJECTNAME__DISKSTORE_MXBEAN =
       OBJECTNAME__PREFIX + "service=DiskStore,name={0},type=Member,member={1}";
 
-  public static final String OBJECTNAME__LOCATOR_MXBEAN =
-      OBJECTNAME__PREFIX + "service=Locator,type=Member,member={0}";
+  String OBJECTNAME__LOCATOR_MXBEAN = OBJECTNAME__PREFIX + "service=Locator,type=Member,member={0}";
 
-  public static final String OBJECTNAME__CACHESERVICE_MXBEAN =
+  String OBJECTNAME__CACHESERVICE_MXBEAN =
       OBJECTNAME__PREFIX + "service=CacheService,name={0},type=Member,member={1}";
 
-  public static final String AGGREGATE_MBEAN_PATTERN = OBJECTNAME__PREFIX + "*,type=Distributed";
+  String AGGREGATE_MBEAN_PATTERN = OBJECTNAME__PREFIX + "*,type=Distributed";
   // Object Name keys
 
-  public static final String OBJECTNAME_MEMBER_APPENDER = "member";
+  String OBJECTNAME_MEMBER_APPENDER = "member";
 
-  public static final int MAX_SHOW_LOG_LINES = 100;
-  public static final int DEFAULT_SHOW_LOG_LINES = 30;
+  int MAX_SHOW_LOG_LINES = 100;
+  int DEFAULT_SHOW_LOG_LINES = 30;
 
-  public static final String GATEWAY_SENDER_PATTERN =
-      OBJECTNAME__PREFIX + "service=GatewaySender,*";
+  String GATEWAY_SENDER_PATTERN = OBJECTNAME__PREFIX + "service=GatewaySender,*";
 
-  public static final String NOTIFICATION_HUB_LISTENER = "GemFire:service=NotificationHubListener";
+  String NOTIFICATION_HUB_LISTENER = "GemFire:service=NotificationHubListener";
 
-  public static final String LINUX_SYSTEM = "Linux";
+  String LINUX_SYSTEM = "Linux";
 
   /**
    * Factor converting bytes to MB
    */
-  public static final long MBFactor = 1024 * 1024;
+  long MBFactor = 1024 * 1024;
 
-  public static final String PULSE_URL = "http://{0}:{1}/pulse";
+  String PULSE_URL = "http://{0}:{1}/pulse";
 
-  public static final String DEFAULT_HOST_NAME = "localhost";
+  String DEFAULT_HOST_NAME = "localhost";
 
   int NOTIF_REGION_MAX_ENTRIES = 10;
 
 
 
-  public static final String REGION_CREATED_PREFIX = "Region Created With Name ";
-  public static final String REGION_CLOSED_PREFIX = "Region Destroyed/Closed With Name ";
+  String REGION_CREATED_PREFIX = "Region Created With Name ";
+  String REGION_CLOSED_PREFIX = "Region Destroyed/Closed With Name ";
 
-  public static final String DISK_STORE_CREATED_PREFIX = "DiskStore Created With Name ";
-  public static final String DISK_STORE_CLOSED_PREFIX = "DiskStore Destroyed/Closed With Name ";
+  String DISK_STORE_CREATED_PREFIX = "DiskStore Created With Name ";
+  String DISK_STORE_CLOSED_PREFIX = "DiskStore Destroyed/Closed With Name ";
 
-  public static final String LOCK_SERVICE_CREATED_PREFIX = "LockService Created With Name ";
-  public static final String LOCK_SERVICE_CLOSED_PREFIX = "Lockservice closed With Name ";
+  String LOCK_SERVICE_CREATED_PREFIX = "LockService Created With Name ";
+  String LOCK_SERVICE_CLOSED_PREFIX = "Lockservice closed With Name ";
 
-  public static final String CACHE_MEMBER_DEPARTED_PREFIX = "Member Departed ";
-  public static final String CACHE_MEMBER_JOINED_PREFIX = "Member Joined ";
-  public static final String CACHE_MEMBER_SUSPECT_PREFIX = "Member Suspected ";
+  String CACHE_MEMBER_DEPARTED_PREFIX = "Member Departed ";
+  String CACHE_MEMBER_JOINED_PREFIX = "Member Joined ";
+  String CACHE_MEMBER_SUSPECT_PREFIX = "Member Suspected ";
 
-  public static final String GATEWAY_SENDER_CREATED_PREFIX = "GatewaySender Created in the VM ";
-  public static final String GATEWAY_SENDER_STARTED_PREFIX = "GatewaySender Started in the VM ";
-  public static final String GATEWAY_SENDER_STOPPED_PREFIX = "GatewaySender Stopped in the VM ";
-  public static final String GATEWAY_SENDER_PAUSED_PREFIX = "GatewaySender Paused in the VM ";
-  public static final String GATEWAY_SENDER_RESUMED_PREFIX = "GatewaySender Resumed in the VM ";
+  String GATEWAY_SENDER_CREATED_PREFIX = "GatewaySender Created in the VM ";
+  String GATEWAY_SENDER_STARTED_PREFIX = "GatewaySender Started in the VM ";
+  String GATEWAY_SENDER_STOPPED_PREFIX = "GatewaySender Stopped in the VM ";
+  String GATEWAY_SENDER_PAUSED_PREFIX = "GatewaySender Paused in the VM ";
+  String GATEWAY_SENDER_RESUMED_PREFIX = "GatewaySender Resumed in the VM ";
 
-  public static final String GATEWAY_RECEIVER_CREATED_PREFIX = "GatewayReceiver Created in the VM ";
-  public static final String GATEWAY_RECEIVER_STARTED_PREFIX = "GatewayReceiver Started in the VM ";
-  public static final String GATEWAY_RECEIVER_STOPPED_PREFIX = "GatewayReceiver Stopped in the VM ";
+  String GATEWAY_RECEIVER_CREATED_PREFIX = "GatewayReceiver Created in the VM ";
+  String GATEWAY_RECEIVER_STARTED_PREFIX = "GatewayReceiver Started in the VM ";
+  String GATEWAY_RECEIVER_STOPPED_PREFIX = "GatewayReceiver Stopped in the VM ";
 
-  public static final String ASYNC_EVENT_QUEUE_CREATED_PREFIX =
-      "Async Event Queue is Created  in the VM ";
-  public static final String ASYNC_EVENT_QUEUE_CLOSED_PREFIX =
-      "Async Event Queue is Closed in the VM ";
+  String ASYNC_EVENT_QUEUE_CREATED_PREFIX = "Async Event Queue is Created  in the VM ";
+  String ASYNC_EVENT_QUEUE_CLOSED_PREFIX = "Async Event Queue is Closed in the VM ";
 
-  public static final String CACHE_SERVICE_CREATED_PREFIX = "Cache Service Created With Name ";
+  String CACHE_SERVICE_CREATED_PREFIX = "Cache Service Created With Name ";
 
-  public static final String CACHE_SERVER_STARTED_PREFIX = "Cache Server is Started in the VM ";
-  public static final String CACHE_SERVER_STOPPED_PREFIX = "Cache Server is stopped in the VM ";
+  String CACHE_SERVER_STARTED_PREFIX = "Cache Server is Started in the VM ";
+  String CACHE_SERVER_STOPPED_PREFIX = "Cache Server is stopped in the VM ";
 
-  public static final String CLIENT_JOINED_PREFIX = "Client joined with Id ";
-  public static final String CLIENT_CRASHED_PREFIX = "Client crashed with Id ";
-  public static final String CLIENT_LEFT_PREFIX = "Client left with Id ";
+  String CLIENT_JOINED_PREFIX = "Client joined with Id ";
+  String CLIENT_CRASHED_PREFIX = "Client crashed with Id ";
+  String CLIENT_LEFT_PREFIX = "Client left with Id ";
 
-  public static final String LOCATOR_STARTED_PREFIX = "Locator is Started in the VM ";
+  String LOCATOR_STARTED_PREFIX = "Locator is Started in the VM ";
 
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/NotificationBroadCasterProxy.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/NotificationBroadCasterProxy.java b/geode-core/src/main/java/org/apache/geode/management/internal/NotificationBroadCasterProxy.java
index 16215ac8..3cbb818 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/NotificationBroadCasterProxy.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/NotificationBroadCasterProxy.java
@@ -32,5 +32,5 @@ public interface NotificationBroadCasterProxy extends NotificationBroadcaster {
    * 
    * @param notification
    */
-  public void sendNotification(Notification notification);
+  void sendNotification(Notification notification);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/ProxyInterface.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/ProxyInterface.java b/geode-core/src/main/java/org/apache/geode/management/internal/ProxyInterface.java
index ac3408e..8da434f 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/ProxyInterface.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/ProxyInterface.java
@@ -26,14 +26,14 @@ public interface ProxyInterface {
    * 
    * @return last refreshed time
    */
-  public long getLastRefreshedTime();
+  long getLastRefreshedTime();
 
   /**
    * Sets the last refreshed time for the proxy
    * 
    * @param lastRefreshedTime
    */
-  public void setLastRefreshedTime(long lastRefreshedTime);
+  void setLastRefreshedTime(long lastRefreshedTime);
 
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/ProxyListener.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/ProxyListener.java b/geode-core/src/main/java/org/apache/geode/management/internal/ProxyListener.java
index f96bbec..6bdfddc 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/ProxyListener.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/ProxyListener.java
@@ -91,7 +91,7 @@ public interface ProxyListener extends MembershipListener {
    * 
    * @param notification
    */
-  public void handleNotification(Notification notification);
+  void handleNotification(Notification notification);
 
   /**
    * This is a very special method, Only to be invoked from MonitoringRegionCacheListener. The need
@@ -108,7 +108,7 @@ public interface ProxyListener extends MembershipListener {
    *        interface class type.
    * @param newVal new value for the proxy
    */
-  public void afterPseudoCreateProxy(ObjectName objectName, Class interfaceClass,
-      Object proxyObject, FederationComponent newVal);
+  void afterPseudoCreateProxy(ObjectName objectName, Class interfaceClass, Object proxyObject,
+      FederationComponent newVal);
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/beans/AggregateHandler.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/beans/AggregateHandler.java b/geode-core/src/main/java/org/apache/geode/management/internal/beans/AggregateHandler.java
index 61c8f09..8a7b892 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/beans/AggregateHandler.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/beans/AggregateHandler.java
@@ -34,7 +34,7 @@ public interface AggregateHandler {
    * @param proxyObject actual reference of the proxy.
    * @param newVal new value of the Proxy
    */
-  public void handleProxyAddition(ObjectName objectName, Class interfaceClass, Object proxyObject,
+  void handleProxyAddition(ObjectName objectName, Class interfaceClass, Object proxyObject,
       FederationComponent newVal);
 
   /**
@@ -44,7 +44,7 @@ public interface AggregateHandler {
    * @param proxyObject actual reference of the proxy.
    * @param oldVal old value of the Proxy
    */
-  public void handleProxyRemoval(ObjectName objectName, Class interfaceClass, Object proxyObject,
+  void handleProxyRemoval(ObjectName objectName, Class interfaceClass, Object proxyObject,
       FederationComponent oldVal);
 
   /**
@@ -55,7 +55,7 @@ public interface AggregateHandler {
    * @param newVal new value of the Proxy
    * @param oldVal old value of the proxy
    */
-  public void handleProxyUpdate(ObjectName objectName, Class interfaceClass, Object proxyObject,
+  void handleProxyUpdate(ObjectName objectName, Class interfaceClass, Object proxyObject,
       FederationComponent newVal, FederationComponent oldVal);
 
   /**
@@ -65,7 +65,7 @@ public interface AggregateHandler {
    * @param proxyObject actual reference of the proxy.
    * @param newVal new value of the Proxy
    */
-  public void handlePseudoCreateProxy(ObjectName objectName, Class interfaceClass,
-      Object proxyObject, FederationComponent newVal);
+  void handlePseudoCreateProxy(ObjectName objectName, Class interfaceClass, Object proxyObject,
+      FederationComponent newVal);
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/beans/CacheServiceMBeanBase.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/beans/CacheServiceMBeanBase.java b/geode-core/src/main/java/org/apache/geode/management/internal/beans/CacheServiceMBeanBase.java
index 1ef7699..abaeb20 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/beans/CacheServiceMBeanBase.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/beans/CacheServiceMBeanBase.java
@@ -16,7 +16,7 @@ package org.apache.geode.management.internal.beans;
 
 public interface CacheServiceMBeanBase {
 
-  public String getId();
+  String getId();
 
-  public Class getInterfaceClass();
+  Class getInterfaceClass();
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/cli/multistep/CLIStep.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/multistep/CLIStep.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/multistep/CLIStep.java
index a037f51..a515bf1 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/multistep/CLIStep.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/multistep/CLIStep.java
@@ -24,8 +24,8 @@ import org.apache.geode.management.cli.Result;
  */
 public interface CLIStep {
 
-  public Result exec();
+  Result exec();
 
-  public String getName();
+  String getName();
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/cli/result/ResultData.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/result/ResultData.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/result/ResultData.java
index 37a1d60..3d8c9a8 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/result/ResultData.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/result/ResultData.java
@@ -44,7 +44,7 @@ public interface ResultData {
 
   String getType();
 
-  public Status getStatus();
+  Status getStatus();
 
-  public void setStatus(final Status status);
+  void setStatus(final Status status);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/OperationInvoker.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/OperationInvoker.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/OperationInvoker.java
index 19b5081..ac5449b 100755
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/OperationInvoker.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/OperationInvoker.java
@@ -43,7 +43,7 @@ public interface OperationInvoker {
    * @return true if there is an active connection, false otherwise.
    * @see #isReady()
    */
-  public boolean isConnected();
+  boolean isConnected();
 
   /**
    * Determines whether there is an active, current active connection ready for use.
@@ -52,7 +52,7 @@ public interface OperationInvoker {
    * @return true if this {@linkplain OperationInvoker} is ready for operation, false otherwise.
    * @see #isConnected()
    */
-  public boolean isReady();
+  boolean isReady();
 
   /**
    * Read the attribute identified by name from a remote resource identified by name.
@@ -62,7 +62,7 @@ public interface OperationInvoker {
    * @param attributeName name of the attribute to be fetched.
    * @return value of the named attribute on the remote resource.
    */
-  public Object getAttribute(String resourceName, String attributeName);
+  Object getAttribute(String resourceName, String attributeName);
 
   /**
    * Gets the identifier of the GemFire cluster.
@@ -70,7 +70,7 @@ public interface OperationInvoker {
    * 
    * @return an integer value indicating the identifier of the GemFire cluster.
    */
-  public int getClusterId();
+  int getClusterId();
 
   // TODO continue to add MXBean accessor methods as necessary for GemFire MXBeans used in Gfsh and
   // command classes...
@@ -82,7 +82,7 @@ public interface OperationInvoker {
    * @return a proxy instance of the GemFire Manager's DistributedSystem MXBean.
    * @see org.apache.geode.management.DistributedSystemMXBean
    */
-  public DistributedSystemMXBean getDistributedSystemMXBean();
+  DistributedSystemMXBean getDistributedSystemMXBean();
 
   /**
    * Gets a proxy to an MXBean on a remote MBeanServer.
@@ -95,7 +95,7 @@ public interface OperationInvoker {
    * @return a proxy to access the specified, remote MXBean.
    * @see javax.management.ObjectName
    */
-  public <T> T getMBeanProxy(ObjectName objectName, Class<T> mbeanInterface);
+  <T> T getMBeanProxy(ObjectName objectName, Class<T> mbeanInterface);
 
   /**
    * Invoke an operation identified by name on a remote resource identified by name with the given
@@ -109,8 +109,7 @@ public interface OperationInvoker {
    * @param signature an array containing the signature of the operation.
    * @return result of the operation invocation.
    */
-  public Object invoke(String resourceName, String operationName, Object[] params,
-      String[] signature);
+  Object invoke(String resourceName, String operationName, Object[] params, String[] signature);
 
   /**
    * This method searches the MBean server, based on the OperationsInvoker's JMX-based or other
@@ -126,7 +125,7 @@ public interface OperationInvoker {
    * @see javax.management.ObjectName
    * @see javax.management.QueryExp
    */
-  public Set<ObjectName> queryNames(ObjectName objectName, QueryExp queryExpression);
+  Set<ObjectName> queryNames(ObjectName objectName, QueryExp queryExpression);
 
   /**
    * Processes the requested command. Sends the command to the GemFire Manager for remote processing
@@ -137,11 +136,11 @@ public interface OperationInvoker {
    * @return the result of the command execution.
    * @see org.apache.geode.management.internal.cli.CommandRequest
    */
-  public Object processCommand(CommandRequest command);
+  Object processCommand(CommandRequest command);
 
   /**
    * Stops this {@linkplain OperationInvoker}
    */
-  public void stop();
+  void stop();
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/internal/security/AccessControlMXBean.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/security/AccessControlMXBean.java b/geode-core/src/main/java/org/apache/geode/management/internal/security/AccessControlMXBean.java
index b6c42ed..15c908b 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/security/AccessControlMXBean.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/security/AccessControlMXBean.java
@@ -21,6 +21,6 @@ package org.apache.geode.management.internal.security;
  */
 public interface AccessControlMXBean {
 
-  public boolean authorize(String resource, String permission);
+  boolean authorize(String resource, String permission);
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipEvent.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipEvent.java b/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipEvent.java
index 5eb3691..535a0a9 100644
--- a/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipEvent.java
+++ b/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipEvent.java
@@ -29,19 +29,19 @@ public interface ClientMembershipEvent {
    *
    * @see org.apache.geode.distributed.DistributedSystem#getDistributedMember
    */
-  public DistributedMember getMember();
+  DistributedMember getMember();
 
   /**
    * Returns the id of the member that connected or disconnected.
    *
    * @see org.apache.geode.distributed.DistributedMember#getId
    */
-  public String getMemberId();
+  String getMemberId();
 
   /**
    * Returns true if the member is a client to a CacheServer hosted by this process. Returns false
    * if the member is a peer that this process is connected to.
    */
-  public boolean isClient();
+  boolean isClient();
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipListener.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipListener.java b/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipListener.java
index 313f501..a248284 100644
--- a/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipListener.java
+++ b/geode-core/src/main/java/org/apache/geode/management/membership/ClientMembershipListener.java
@@ -28,18 +28,18 @@ public interface ClientMembershipListener {
    * Invoked when a client has connected to this process or when this process has connected to a
    * CacheServer.
    */
-  public void memberJoined(ClientMembershipEvent event);
+  void memberJoined(ClientMembershipEvent event);
 
   /**
    * Invoked when a client has gracefully disconnected from this process or when this process has
    * gracefully disconnected from a CacheServer.
    */
-  public void memberLeft(ClientMembershipEvent event);
+  void memberLeft(ClientMembershipEvent event);
 
   /**
    * Invoked when a client has unexpectedly disconnected from this process or when this process has
    * unexpectedly disconnected from a CacheServer.
    */
-  public void memberCrashed(ClientMembershipEvent event);
+  void memberCrashed(ClientMembershipEvent event);
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/membership/MembershipEvent.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/membership/MembershipEvent.java b/geode-core/src/main/java/org/apache/geode/management/membership/MembershipEvent.java
index 12256eb..aa66bb1 100644
--- a/geode-core/src/main/java/org/apache/geode/management/membership/MembershipEvent.java
+++ b/geode-core/src/main/java/org/apache/geode/management/membership/MembershipEvent.java
@@ -26,7 +26,7 @@ public interface MembershipEvent {
   /**
    * Returns the distributed member as a String.
    */
-  public String getMemberId();
+  String getMemberId();
 
   /**
    * Returns the {@link DistributedMember} that this event originated in.
@@ -34,5 +34,5 @@ public interface MembershipEvent {
    * @return the member that performed the operation that originated this event.
    * @since GemFire 8.0
    */
-  public DistributedMember getDistributedMember();
+  DistributedMember getDistributedMember();
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/management/membership/MembershipListener.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/membership/MembershipListener.java b/geode-core/src/main/java/org/apache/geode/management/membership/MembershipListener.java
index d9fa97f..fd0ef0f 100644
--- a/geode-core/src/main/java/org/apache/geode/management/membership/MembershipListener.java
+++ b/geode-core/src/main/java/org/apache/geode/management/membership/MembershipListener.java
@@ -27,13 +27,13 @@ public interface MembershipListener {
   /**
    * Invoked when a member has joined the distributed system
    */
-  public void memberJoined(MembershipEvent event);
+  void memberJoined(MembershipEvent event);
 
   /**
    * Invoked when a member has gracefully left the distributed system. This occurs when the member
    * took action to remove itself from the distributed system.
    */
-  public void memberLeft(MembershipEvent event);
+  void memberLeft(MembershipEvent event);
 
   /**
    * Invoked when a member has unexpectedly left the distributed system. This occurs when a member
@@ -42,6 +42,6 @@ public interface MembershipListener {
    * detection</a>, or <a href=../distributed/DistributedSystem.html#enable-network
    * -partition-detection> network partition detection</a> processing.
    */
-  public void memberCrashed(MembershipEvent event);
+  void memberCrashed(MembershipEvent event);
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/PdxInstance.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/PdxInstance.java b/geode-core/src/main/java/org/apache/geode/pdx/PdxInstance.java
index a463a86..eca9f7a 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/PdxInstance.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/PdxInstance.java
@@ -51,7 +51,7 @@ public interface PdxInstance extends java.io.Serializable {
    * @return the name of the class that this pdx instance represents.
    * @since GemFire 6.6.2
    */
-  public String getClassName();
+  String getClassName();
 
   /**
    * Returns true if this instance represents an enum. Enum's have a String field named "name" and
@@ -60,7 +60,7 @@ public interface PdxInstance extends java.io.Serializable {
    * 
    * @return true if this instance represents an enum.
    */
-  public boolean isEnum();
+  boolean isEnum();
 
   /**
    * Deserializes and returns the domain object that this instance represents.
@@ -68,7 +68,7 @@ public interface PdxInstance extends java.io.Serializable {
    * @return the deserialized domain object.
    * @throws PdxSerializationException if the instance could not be deserialized
    */
-  public Object getObject();
+  Object getObject();
 
   /**
    * Checks if the named field exists and returns the result.
@@ -78,14 +78,14 @@ public interface PdxInstance extends java.io.Serializable {
    * @param fieldName the name of the field to check
    * @return <code>true</code> if the named field exists; otherwise <code>false</code>
    */
-  public boolean hasField(String fieldName);
+  boolean hasField(String fieldName);
 
   /**
    * Return an unmodifiable list of the field names on this PdxInstance.
    * 
    * @return an unmodifiable list of the field names on this PdxInstance
    */
-  public List<String> getFieldNames();
+  List<String> getFieldNames();
 
   /**
    * Checks if the named field was {@link PdxWriter#markIdentityField(String) marked} as an identity
@@ -98,7 +98,7 @@ public interface PdxInstance extends java.io.Serializable {
    * @return <code>true</code> if the named field exists and was marked as an identify field;
    *         otherwise <code>false</code>
    */
-  public boolean isIdentityField(String fieldName);
+  boolean isIdentityField(String fieldName);
 
   /**
    * Reads the named field and returns its value. If the field does not exist <code>null</code> is
@@ -118,7 +118,7 @@ public interface PdxInstance extends java.io.Serializable {
    *         <code>null</code> is returned.
    * @throws PdxSerializationException if the field could not be deserialized
    */
-  public Object getField(String fieldName);
+  Object getField(String fieldName);
 
   /**
    * Returns true if the given object is equals to this instance.
@@ -166,7 +166,7 @@ public interface PdxInstance extends java.io.Serializable {
    * @param other the other instance to compare to this.
    * @return <code>true</code> if this instance is equal to <code>other</code>.
    */
-  public boolean equals(Object other);
+  boolean equals(Object other);
 
   /**
    * Generates a hashCode based on the identity fields of this PdxInstance.
@@ -190,7 +190,7 @@ public interface PdxInstance extends java.io.Serializable {
    * = hashCode*31 + Arrays.deepHashCode(field); } else { hashCode = hashCode*31 + field.hashCode();
    * } } if (hashCode == 0) { hashCode = 1; }
    */
-  public int hashCode();
+  int hashCode();
 
   /**
    * Prints out all of the identity fields of this PdxInstance.
@@ -199,7 +199,7 @@ public interface PdxInstance extends java.io.Serializable {
    * markIdentityField} then only the marked identity fields are its identity fields. Otherwise all
    * its fields are identity fields.
    */
-  public String toString();
+  String toString();
 
   /**
    * Creates and returns a {@link WritablePdxInstance} whose initial values are those of this
@@ -209,6 +209,6 @@ public interface PdxInstance extends java.io.Serializable {
    * @return a {@link WritablePdxInstance}
    * @throws IllegalStateException if the PdxInstance is an enum.
    */
-  public WritablePdxInstance createWriter();
+  WritablePdxInstance createWriter();
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/PdxInstanceFactory.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/PdxInstanceFactory.java b/geode-core/src/main/java/org/apache/geode/pdx/PdxInstanceFactory.java
index 2214456..41d7ffd 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/PdxInstanceFactory.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/PdxInstanceFactory.java
@@ -36,7 +36,7 @@ public interface PdxInstanceFactory {
    * @return the created instance
    * @throws IllegalStateException if called more than once
    */
-  public PdxInstance create();
+  PdxInstance create();
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -50,7 +50,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeChar(String fieldName, char value);
+  PdxInstanceFactory writeChar(String fieldName, char value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -64,7 +64,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeBoolean(String fieldName, boolean value);
+  PdxInstanceFactory writeBoolean(String fieldName, boolean value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -78,7 +78,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeByte(String fieldName, byte value);
+  PdxInstanceFactory writeByte(String fieldName, byte value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -92,7 +92,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeShort(String fieldName, short value);
+  PdxInstanceFactory writeShort(String fieldName, short value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -106,7 +106,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeInt(String fieldName, int value);
+  PdxInstanceFactory writeInt(String fieldName, int value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -120,7 +120,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeLong(String fieldName, long value);
+  PdxInstanceFactory writeLong(String fieldName, long value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -134,7 +134,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeFloat(String fieldName, float value);
+  PdxInstanceFactory writeFloat(String fieldName, float value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -148,7 +148,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeDouble(String fieldName, double value);
+  PdxInstanceFactory writeDouble(String fieldName, double value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -162,7 +162,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeDate(String fieldName, Date value);
+  PdxInstanceFactory writeDate(String fieldName, Date value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -176,7 +176,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeString(String fieldName, String value);
+  PdxInstanceFactory writeString(String fieldName, String value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -196,7 +196,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeObject(String fieldName, Object value);
+  PdxInstanceFactory writeObject(String fieldName, Object value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -254,7 +254,7 @@ public interface PdxInstanceFactory {
    * @throws PdxSerializationException if serialization of the field fails.
    * @since GemFire 6.6.2
    */
-  public PdxInstanceFactory writeObject(String fieldName, Object value, boolean checkPortability);
+  PdxInstanceFactory writeObject(String fieldName, Object value, boolean checkPortability);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -268,7 +268,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeBooleanArray(String fieldName, boolean[] value);
+  PdxInstanceFactory writeBooleanArray(String fieldName, boolean[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -282,7 +282,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeCharArray(String fieldName, char[] value);
+  PdxInstanceFactory writeCharArray(String fieldName, char[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -296,7 +296,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeByteArray(String fieldName, byte[] value);
+  PdxInstanceFactory writeByteArray(String fieldName, byte[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -310,7 +310,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeShortArray(String fieldName, short[] value);
+  PdxInstanceFactory writeShortArray(String fieldName, short[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -324,7 +324,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeIntArray(String fieldName, int[] value);
+  PdxInstanceFactory writeIntArray(String fieldName, int[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -338,7 +338,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeLongArray(String fieldName, long[] value);
+  PdxInstanceFactory writeLongArray(String fieldName, long[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -352,7 +352,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeFloatArray(String fieldName, float[] value);
+  PdxInstanceFactory writeFloatArray(String fieldName, float[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -366,7 +366,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeDoubleArray(String fieldName, double[] value);
+  PdxInstanceFactory writeDoubleArray(String fieldName, double[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -380,7 +380,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeStringArray(String fieldName, String[] value);
+  PdxInstanceFactory writeStringArray(String fieldName, String[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -398,7 +398,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeObjectArray(String fieldName, Object[] value);
+  PdxInstanceFactory writeObjectArray(String fieldName, Object[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -421,8 +421,7 @@ public interface PdxInstanceFactory {
    * @throws PdxSerializationException if serialization of the field fails.
    * @since GemFire 6.6.2
    */
-  public PdxInstanceFactory writeObjectArray(String fieldName, Object[] value,
-      boolean checkPortability);
+  PdxInstanceFactory writeObjectArray(String fieldName, Object[] value, boolean checkPortability);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -436,7 +435,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxInstanceFactory writeArrayOfByteArrays(String fieldName, byte[][] value);
+  PdxInstanceFactory writeArrayOfByteArrays(String fieldName, byte[][] value);
 
   /**
    * Writes the named field with the given value and type to the serialized form. This method uses
@@ -481,7 +480,7 @@ public interface PdxInstanceFactory {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public <CT, VT extends CT> PdxInstanceFactory writeField(String fieldName, VT fieldValue,
+  <CT, VT extends CT> PdxInstanceFactory writeField(String fieldName, VT fieldValue,
       Class<CT> fieldType);
 
   /**
@@ -531,7 +530,7 @@ public interface PdxInstanceFactory {
    * @throws PdxSerializationException if serialization of the field fails.
    * @since GemFire 6.6.2
    */
-  public <CT, VT extends CT> PdxInstanceFactory writeField(String fieldName, VT fieldValue,
+  <CT, VT extends CT> PdxInstanceFactory writeField(String fieldName, VT fieldValue,
       Class<CT> fieldType, boolean checkPortability);
 
   /**
@@ -552,5 +551,5 @@ public interface PdxInstanceFactory {
    * @return this PdxInstanceFactory
    * @throws PdxFieldDoesNotExistException if the named field has not already been written.
    */
-  public PdxInstanceFactory markIdentityField(String fieldName);
+  PdxInstanceFactory markIdentityField(String fieldName);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/PdxReader.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/PdxReader.java b/geode-core/src/main/java/org/apache/geode/pdx/PdxReader.java
index c426165..0562fb2 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/PdxReader.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/PdxReader.java
@@ -43,7 +43,7 @@ public interface PdxReader {
    *         field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public char readChar(String fieldName);
+  char readChar(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -54,7 +54,7 @@ public interface PdxReader {
    *         <code>boolean</code> field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public boolean readBoolean(String fieldName);
+  boolean readBoolean(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -65,7 +65,7 @@ public interface PdxReader {
    *         field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public byte readByte(String fieldName);
+  byte readByte(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -76,7 +76,7 @@ public interface PdxReader {
    *         field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public short readShort(String fieldName);
+  short readShort(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -87,7 +87,7 @@ public interface PdxReader {
    *         field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public int readInt(String fieldName);
+  int readInt(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -98,7 +98,7 @@ public interface PdxReader {
    *         field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public long readLong(String fieldName);
+  long readLong(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -109,7 +109,7 @@ public interface PdxReader {
    *         field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public float readFloat(String fieldName);
+  float readFloat(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -120,7 +120,7 @@ public interface PdxReader {
    *         <code>double</code> field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public double readDouble(String fieldName);
+  double readDouble(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -131,7 +131,7 @@ public interface PdxReader {
    *         <code>String</code> field.
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public String readString(String fieldName);
+  String readString(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -142,7 +142,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not an
    *         <code>Object</code> field.
    */
-  public Object readObject(String fieldName);
+  Object readObject(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -153,7 +153,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>char[]</code> field.
    */
-  public char[] readCharArray(String fieldName);
+  char[] readCharArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -164,7 +164,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>boolean[]</code> field.
    */
-  public boolean[] readBooleanArray(String fieldName);
+  boolean[] readBooleanArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -175,7 +175,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>byte[]</code> field.
    */
-  public byte[] readByteArray(String fieldName);
+  byte[] readByteArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -186,7 +186,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>short[]</code> field.
    */
-  public short[] readShortArray(String fieldName);
+  short[] readShortArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -197,7 +197,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a <code>int[]</code>
    *         field.
    */
-  public int[] readIntArray(String fieldName);
+  int[] readIntArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -208,7 +208,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>long[]</code> field.
    */
-  public long[] readLongArray(String fieldName);
+  long[] readLongArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -219,7 +219,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>float[]</code> field.
    */
-  public float[] readFloatArray(String fieldName);
+  float[] readFloatArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -230,7 +230,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>double[]</code> field.
    */
-  public double[] readDoubleArray(String fieldName);
+  double[] readDoubleArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -241,7 +241,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>String[]</code> field.
    */
-  public String[] readStringArray(String fieldName);
+  String[] readStringArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -252,7 +252,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>Object[]</code> field.
    */
-  public Object[] readObjectArray(String fieldName);
+  Object[] readObjectArray(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -263,7 +263,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a
    *         <code>byte[][]</code> field.
    */
-  public byte[][] readArrayOfByteArrays(String fieldName);
+  byte[][] readArrayOfByteArrays(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -274,7 +274,7 @@ public interface PdxReader {
    * @throws PdxFieldTypeMismatchException if the named field exists and is not a <code>Date</code>
    *         field.
    */
-  public Date readDate(String fieldName);
+  Date readDate(String fieldName);
 
   /**
    * Checks if the named field exists and returns the result.
@@ -284,7 +284,7 @@ public interface PdxReader {
    * @param fieldName the name of the field to check
    * @return <code>true</code> if the named field exists; otherwise <code>false</code>
    */
-  public boolean hasField(String fieldName);
+  boolean hasField(String fieldName);
 
   /**
    * Checks if the named field was {@link PdxWriter#markIdentityField(String) marked} as an identity
@@ -297,7 +297,7 @@ public interface PdxReader {
    * @return <code>true</code> if the named field exists and was marked as an identify field;
    *         otherwise <code>false</code>
    */
-  public boolean isIdentityField(String fieldName);
+  boolean isIdentityField(String fieldName);
 
   /**
    * Reads the named field and returns its value.
@@ -306,7 +306,7 @@ public interface PdxReader {
    * @return the value of the field if the field exists; otherwise a default value
    * @throws PdxSerializationException if deserialization of the field fails.
    */
-  public Object readField(String fieldName);
+  Object readField(String fieldName);
 
   /**
    * This method returns an object that represents all the unread fields which must be passed to
@@ -320,5 +320,5 @@ public interface PdxReader {
    *
    * @return an object that represents the unread fields.
    */
-  public PdxUnreadFields readUnreadFields();
+  PdxUnreadFields readUnreadFields();
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializable.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializable.java b/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializable.java
index 4098ff7..63aafca 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializable.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializable.java
@@ -56,12 +56,12 @@ public interface PdxSerializable {
    * 
    * @param writer the {@link PdxWriter} to use to write the PDX fields.
    */
-  public void toData(PdxWriter writer);
+  void toData(PdxWriter writer);
 
   /**
    * Deserializes the PDX fields using the given reader.
    * 
    * @param reader the {@link PdxReader} to use to read the PDX fields.
    */
-  public void fromData(PdxReader reader);
+  void fromData(PdxReader reader);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializer.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializer.java b/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializer.java
index 624aa1c..714495b 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializer.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/PdxSerializer.java
@@ -82,7 +82,7 @@ public interface PdxSerializer {
    * @param out the {@link PdxWriter} to use to serialize the object
    * @return <code>true</code> if the method serialized the object; otherwise <code>false</code>
    */
-  public boolean toData(Object o, PdxWriter out);
+  boolean toData(Object o, PdxWriter out);
 
   /**
    * This method is given an class that should be instantiated and deserialized using the given
@@ -93,5 +93,5 @@ public interface PdxSerializer {
    * @return the deserialized object. <code>null</code> indicates that this PdxSerializer does not
    *         know how to deserialize the given class.
    */
-  public Object fromData(Class<?> clazz, PdxReader in);
+  Object fromData(Class<?> clazz, PdxReader in);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/PdxWriter.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/PdxWriter.java b/geode-core/src/main/java/org/apache/geode/pdx/PdxWriter.java
index 47f0e6f..134e99e 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/PdxWriter.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/PdxWriter.java
@@ -47,7 +47,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeChar(String fieldName, char value);
+  PdxWriter writeChar(String fieldName, char value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -61,7 +61,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeBoolean(String fieldName, boolean value);
+  PdxWriter writeBoolean(String fieldName, boolean value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -75,7 +75,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeByte(String fieldName, byte value);
+  PdxWriter writeByte(String fieldName, byte value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -89,7 +89,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeShort(String fieldName, short value);
+  PdxWriter writeShort(String fieldName, short value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -103,7 +103,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeInt(String fieldName, int value);
+  PdxWriter writeInt(String fieldName, int value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -117,7 +117,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeLong(String fieldName, long value);
+  PdxWriter writeLong(String fieldName, long value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -131,7 +131,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeFloat(String fieldName, float value);
+  PdxWriter writeFloat(String fieldName, float value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -145,7 +145,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeDouble(String fieldName, double value);
+  PdxWriter writeDouble(String fieldName, double value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -159,7 +159,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeDate(String fieldName, Date value);
+  PdxWriter writeDate(String fieldName, Date value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -173,7 +173,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeString(String fieldName, String value);
+  PdxWriter writeString(String fieldName, String value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -193,7 +193,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeObject(String fieldName, Object value);
+  PdxWriter writeObject(String fieldName, Object value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -251,7 +251,7 @@ public interface PdxWriter {
    * @throws PdxSerializationException if serialization of the field fails.
    * @since GemFire 6.6.2
    */
-  public PdxWriter writeObject(String fieldName, Object value, boolean checkPortability);
+  PdxWriter writeObject(String fieldName, Object value, boolean checkPortability);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -265,7 +265,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeBooleanArray(String fieldName, boolean[] value);
+  PdxWriter writeBooleanArray(String fieldName, boolean[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -279,7 +279,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeCharArray(String fieldName, char[] value);
+  PdxWriter writeCharArray(String fieldName, char[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -293,7 +293,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeByteArray(String fieldName, byte[] value);
+  PdxWriter writeByteArray(String fieldName, byte[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -307,7 +307,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeShortArray(String fieldName, short[] value);
+  PdxWriter writeShortArray(String fieldName, short[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -321,7 +321,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeIntArray(String fieldName, int[] value);
+  PdxWriter writeIntArray(String fieldName, int[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -335,7 +335,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeLongArray(String fieldName, long[] value);
+  PdxWriter writeLongArray(String fieldName, long[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -349,7 +349,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeFloatArray(String fieldName, float[] value);
+  PdxWriter writeFloatArray(String fieldName, float[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -363,7 +363,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeDoubleArray(String fieldName, double[] value);
+  PdxWriter writeDoubleArray(String fieldName, double[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -377,7 +377,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeStringArray(String fieldName, String[] value);
+  PdxWriter writeStringArray(String fieldName, String[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -395,7 +395,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeObjectArray(String fieldName, Object[] value);
+  PdxWriter writeObjectArray(String fieldName, Object[] value);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -418,7 +418,7 @@ public interface PdxWriter {
    * @throws PdxSerializationException if serialization of the field fails.
    * @since GemFire 6.6.2
    */
-  public PdxWriter writeObjectArray(String fieldName, Object[] value, boolean checkPortability);
+  PdxWriter writeObjectArray(String fieldName, Object[] value, boolean checkPortability);
 
   /**
    * Writes the named field with the given value to the serialized form. The fields type is
@@ -432,7 +432,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public PdxWriter writeArrayOfByteArrays(String fieldName, byte[][] value);
+  PdxWriter writeArrayOfByteArrays(String fieldName, byte[][] value);
 
   /**
    * Writes the named field with the given value and type to the serialized form. This method uses
@@ -477,8 +477,7 @@ public interface PdxWriter {
    * @throws PdxFieldAlreadyExistsException if the named field has already been written
    * @throws PdxSerializationException if serialization of the field fails.
    */
-  public <CT, VT extends CT> PdxWriter writeField(String fieldName, VT fieldValue,
-      Class<CT> fieldType);
+  <CT, VT extends CT> PdxWriter writeField(String fieldName, VT fieldValue, Class<CT> fieldType);
 
   /**
    * Writes the named field with the given value and type to the serialized form. This method uses
@@ -528,8 +527,8 @@ public interface PdxWriter {
    * @throws PdxSerializationException if serialization of the field fails.
    * @since GemFire 6.6.2
    */
-  public <CT, VT extends CT> PdxWriter writeField(String fieldName, VT fieldValue,
-      Class<CT> fieldType, boolean checkPortability);
+  <CT, VT extends CT> PdxWriter writeField(String fieldName, VT fieldValue, Class<CT> fieldType,
+      boolean checkPortability);
 
   /**
    * Writes the given unread fields to the serialized form. The unread fields are obtained by
@@ -542,7 +541,7 @@ public interface PdxWriter {
    * @return this PdxWriter
    * @throws PdxFieldAlreadyExistsException if one of the writeXXX methods has already been called.
    */
-  public PdxWriter writeUnreadFields(PdxUnreadFields unread);
+  PdxWriter writeUnreadFields(PdxUnreadFields unread);
 
   /**
    * Indicate that the named field should be included in hashCode and equals checks of this object
@@ -562,5 +561,5 @@ public interface PdxWriter {
    * @return this PdxWriter
    * @throws PdxFieldDoesNotExistException if the named field has not already been written.
    */
-  public PdxWriter markIdentityField(String fieldName);
+  PdxWriter markIdentityField(String fieldName);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/WritablePdxInstance.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/WritablePdxInstance.java b/geode-core/src/main/java/org/apache/geode/pdx/WritablePdxInstance.java
index 0105ecc..22680bb 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/WritablePdxInstance.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/WritablePdxInstance.java
@@ -34,5 +34,5 @@ public interface WritablePdxInstance extends PdxInstance {
    * @throws PdxFieldDoesNotExistException if the named field does not exist
    * @throws PdxFieldTypeMismatchException if the type of the value is not compatible with the field
    */
-  public void setField(String fieldName, Object value);
+  void setField(String fieldName, Object value);
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/internal/ComparableEnum.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/internal/ComparableEnum.java b/geode-core/src/main/java/org/apache/geode/pdx/internal/ComparableEnum.java
index 4783716..feb7446 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/internal/ComparableEnum.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/internal/ComparableEnum.java
@@ -16,9 +16,9 @@ package org.apache.geode.pdx.internal;
 
 @SuppressWarnings("rawtypes")
 public interface ComparableEnum extends Comparable {
-  public String getClassName();
+  String getClassName();
 
-  public String getName();
+  String getName();
 
-  public int getOrdinal();
+  int getOrdinal();
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/internal/ConvertableToBytes.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/internal/ConvertableToBytes.java b/geode-core/src/main/java/org/apache/geode/pdx/internal/ConvertableToBytes.java
index f5cbe66..3a4453f 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/internal/ConvertableToBytes.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/internal/ConvertableToBytes.java
@@ -17,5 +17,5 @@ package org.apache.geode.pdx.internal;
 import java.io.IOException;
 
 public interface ConvertableToBytes {
-  public byte[] toBytes() throws IOException;
+  byte[] toBytes() throws IOException;
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/internal/InternalPdxReader.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/internal/InternalPdxReader.java b/geode-core/src/main/java/org/apache/geode/pdx/internal/InternalPdxReader.java
index 7b4ec7e..2ce3565 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/internal/InternalPdxReader.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/internal/InternalPdxReader.java
@@ -27,98 +27,98 @@ import org.apache.geode.pdx.internal.AutoSerializableManager.AutoClassInfo;
  * @since GemFire 6.6.2
  */
 public interface InternalPdxReader extends PdxReader {
-  public PdxField getPdxField(String fieldName);
+  PdxField getPdxField(String fieldName);
 
-  public char readChar(PdxField f);
+  char readChar(PdxField f);
 
-  public boolean readBoolean(PdxField f);
+  boolean readBoolean(PdxField f);
 
-  public byte readByte(PdxField f);
+  byte readByte(PdxField f);
 
-  public short readShort(PdxField f);
+  short readShort(PdxField f);
 
-  public int readInt(PdxField f);
+  int readInt(PdxField f);
 
-  public long readLong(PdxField f);
+  long readLong(PdxField f);
 
-  public float readFloat(PdxField f);
+  float readFloat(PdxField f);
 
-  public double readDouble(PdxField f);
+  double readDouble(PdxField f);
 
-  public String readString(PdxField f);
+  String readString(PdxField f);
 
-  public Object readObject(PdxField f);
+  Object readObject(PdxField f);
 
-  public char[] readCharArray(PdxField f);
+  char[] readCharArray(PdxField f);
 
-  public boolean[] readBooleanArray(PdxField f);
+  boolean[] readBooleanArray(PdxField f);
 
-  public byte[] readByteArray(PdxField f);
+  byte[] readByteArray(PdxField f);
 
-  public short[] readShortArray(PdxField f);
+  short[] readShortArray(PdxField f);
 
-  public int[] readIntArray(PdxField f);
+  int[] readIntArray(PdxField f);
 
-  public long[] readLongArray(PdxField f);
+  long[] readLongArray(PdxField f);
 
-  public float[] readFloatArray(PdxField f);
+  float[] readFloatArray(PdxField f);
 
-  public double[] readDoubleArray(PdxField f);
+  double[] readDoubleArray(PdxField f);
 
-  public String[] readStringArray(PdxField f);
+  String[] readStringArray(PdxField f);
 
-  public Object[] readObjectArray(PdxField f);
+  Object[] readObjectArray(PdxField f);
 
-  public byte[][] readArrayOfByteArrays(PdxField f);
+  byte[][] readArrayOfByteArrays(PdxField f);
 
-  public Date readDate(PdxField f);
+  Date readDate(PdxField f);
 
-  public char readChar();
+  char readChar();
 
-  public boolean readBoolean();
+  boolean readBoolean();
 
-  public byte readByte();
+  byte readByte();
 
-  public short readShort();
+  short readShort();
 
-  public int readInt();
+  int readInt();
 
-  public long readLong();
+  long readLong();
 
-  public float readFloat();
+  float readFloat();
 
-  public double readDouble();
+  double readDouble();
 
-  public String readString();
+  String readString();
 
-  public Object readObject();
+  Object readObject();
 
-  public char[] readCharArray();
+  char[] readCharArray();
 
-  public boolean[] readBooleanArray();
+  boolean[] readBooleanArray();
 
-  public byte[] readByteArray();
+  byte[] readByteArray();
 
-  public short[] readShortArray();
+  short[] readShortArray();
 
-  public int[] readIntArray();
+  int[] readIntArray();
 
-  public long[] readLongArray();
+  long[] readLongArray();
 
-  public float[] readFloatArray();
+  float[] readFloatArray();
 
-  public double[] readDoubleArray();
+  double[] readDoubleArray();
 
-  public String[] readStringArray();
+  String[] readStringArray();
 
-  public Object[] readObjectArray();
+  Object[] readObjectArray();
 
-  public byte[][] readArrayOfByteArrays();
+  byte[][] readArrayOfByteArrays();
 
-  public Date readDate();
+  Date readDate();
 
-  public PdxType getPdxType();
+  PdxType getPdxType();
 
-  public void orderedDeserialize(Object obj, AutoClassInfo ci);
+  void orderedDeserialize(Object obj, AutoClassInfo ci);
 
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/pdx/internal/TypeRegistration.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/pdx/internal/TypeRegistration.java b/geode-core/src/main/java/org/apache/geode/pdx/internal/TypeRegistration.java
index cb8f948..329edbb 100644
--- a/geode-core/src/main/java/org/apache/geode/pdx/internal/TypeRegistration.java
+++ b/geode-core/src/main/java/org/apache/geode/pdx/internal/TypeRegistration.java
@@ -27,42 +27,42 @@ public interface TypeRegistration {
   /**
    * Define the type in the distributed system
    */
-  public int defineType(PdxType newType);
+  int defineType(PdxType newType);
 
   /**
    * Get the type id from the distributed system
    */
-  public PdxType getType(int typeId);
+  PdxType getType(int typeId);
 
   /**
    * Add a type id that has come from a remote member.
    */
-  public void addRemoteType(int typeId, PdxType type);
+  void addRemoteType(int typeId, PdxType type);
 
-  public void addImportedType(int typeId, PdxType importedType);
+  void addImportedType(int typeId, PdxType importedType);
 
   /**
    * Test hook to get the last allocated type id
    */
-  public int getLastAllocatedTypeId();
+  int getLastAllocatedTypeId();
 
-  public void initialize();
+  void initialize();
 
-  public void gatewaySenderStarted(GatewaySender gatewaySender);
+  void gatewaySenderStarted(GatewaySender gatewaySender);
 
-  public void creatingPersistentRegion();
+  void creatingPersistentRegion();
 
-  public void creatingPool();
+  void creatingPool();
 
-  public int getEnumId(Enum<?> v);
+  int getEnumId(Enum<?> v);
 
-  public void addRemoteEnum(int enumId, EnumInfo newInfo);
+  void addRemoteEnum(int enumId, EnumInfo newInfo);
 
-  public void addImportedEnum(int enumId, EnumInfo importedInfo);
+  void addImportedEnum(int enumId, EnumInfo importedInfo);
 
-  public int defineEnum(EnumInfo newInfo);
+  int defineEnum(EnumInfo newInfo);
 
-  public EnumInfo getEnumById(int enumId);
+  EnumInfo getEnumById(int enumId);
 
   /**
    * Returns the currently defined types.
@@ -85,17 +85,17 @@ public interface TypeRegistration {
    * @param className
    * @return PdxType or null if field not present
    */
-  public PdxType getPdxTypeForField(String fieldName, String className);
+  PdxType getPdxTypeForField(String fieldName, String className);
 
   /*
    * test hook
    */
-  public void testClearRegistry();
+  void testClearRegistry();
 
-  public boolean isClient();
+  boolean isClient();
 
   /**
    * Return the size of the type registry in this member.
    */
-  public int getLocalSize();
+  int getLocalSize();
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/ra/GFConnection.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/ra/GFConnection.java b/geode-core/src/main/java/org/apache/geode/ra/GFConnection.java
index 23547ed..da179c8 100644
--- a/geode-core/src/main/java/org/apache/geode/ra/GFConnection.java
+++ b/geode-core/src/main/java/org/apache/geode/ra/GFConnection.java
@@ -20,5 +20,5 @@ import javax.resource.Referenceable;
 import javax.resource.ResourceException;
 
 public interface GFConnection extends Serializable, Referenceable {
-  public void close() throws ResourceException;
+  void close() throws ResourceException;
 }

http://git-wip-us.apache.org/repos/asf/geode/blob/7c7b577e/geode-core/src/main/java/org/apache/geode/ra/GFConnectionFactory.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/ra/GFConnectionFactory.java b/geode-core/src/main/java/org/apache/geode/ra/GFConnectionFactory.java
index 3f6f942..10bbac6 100644
--- a/geode-core/src/main/java/org/apache/geode/ra/GFConnectionFactory.java
+++ b/geode-core/src/main/java/org/apache/geode/ra/GFConnectionFactory.java
@@ -22,5 +22,5 @@ import javax.resource.ResourceException;
 
 
 public interface GFConnectionFactory extends Referenceable, Serializable {
-  public GFConnection getConnection() throws ResourceException;
+  GFConnection getConnection() throws ResourceException;
 }


Mime
View raw message