Clean up the code

Author: xxlabaza

.codestyle/checkstyle.xml

@@ -318,9 +318,9 @@ limitations under the License.
     <!-- Checks for common coding problems               -->
     <!-- See http://checkstyle.sf.net/config_coding.html -->
     <module name="CovariantEquals"/>
-    <!-- <module name="DeclarationOrder">
+    <module name="DeclarationOrder">
       <property name="ignoreConstructors" value="true"/>
-    </module> -->
+    </module>
     <module name="DefaultComesLast"/>
     <module name="EmptyStatement"/>
     <module name="EqualsAvoidNull"/>

core/pom.xml

@@ -44,13 +44,6 @@ limitations under the License.
         <artifactId>maven-javadoc-plugin</artifactId>
       </plugin>
 
-      <plugin>
-        <artifactId>maven-install-plugin</artifactId>
-      </plugin>
-      <plugin>
-        <artifactId>maven-release-plugin</artifactId>
-      </plugin>
-
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>

core/src/main/java/io/appulse/epmd/java/core/model/response/NodeInfo.java

@@ -50,6 +50,40 @@
 @FieldDefaults(level = PRIVATE)
 public class NodeInfo implements DataSerializable {
 
+  boolean ok;
+
+  Optional<Integer> port;
+
+  Optional<NodeType> type;
+
+  Optional<Protocol> protocol;
+
+  Optional<Version> high;
+
+  Optional<Version> low;
+
+  Optional<String> name;
+
+  public NodeInfo () {
+    port = empty();
+    type = empty();
+    protocol = empty();
+    high = empty();
+    low = empty();
+    name = empty();
+  }
+
+  @Builder
+  public NodeInfo (boolean ok, Integer port, NodeType type, Protocol protocol, Version high, Version low, String name) {
+    this.ok = ok;
+    this.port = ofNullable(port);
+    this.type = ofNullable(type);
+    this.protocol = ofNullable(protocol);
+    this.high = ofNullable(high);
+    this.low = ofNullable(low);
+    this.name = ofNullable(name);
+  }
+
   @Override
   public void write (@NonNull Bytes bytes) {
     if (!ok) {
@@ -86,38 +120,4 @@ public void read (@NonNull Bytes bytes) {
     val length = bytes.getShort();
     name = of(bytes.getString(length, ISO_8859_1));
   }
-
-  boolean ok;
-
-  Optional<Integer> port;
-
-  Optional<NodeType> type;
-
-  Optional<Protocol> protocol;
-
-  Optional<Version> high;
-
-  Optional<Version> low;
-
-  Optional<String> name;
-
-  public NodeInfo () {
-    port = empty();
-    type = empty();
-    protocol = empty();
-    high = empty();
-    low = empty();
-    name = empty();
-  }
-
-  @Builder
-  public NodeInfo (boolean ok, Integer port, NodeType type, Protocol protocol, Version high, Version low, String name) {
-    this.ok = ok;
-    this.port = ofNullable(port);
-    this.type = ofNullable(type);
-    this.protocol = ofNullable(protocol);
-    this.high = ofNullable(high);
-    this.low = ofNullable(low);
-    this.name = ofNullable(name);
-  }
 }