refactoring getter/setters for package action.admin.cluster.node.shutdown #2657

Author: dadoonet

src/main/java/org/elasticsearch/action/admin/cluster/node/shutdown/NodesShutdownResponse.java

@@ -43,20 +43,12 @@ public NodesShutdownResponse(ClusterName clusterName, DiscoveryNode[] nodes) {
         this.nodes = nodes;
     }
 
-    public ClusterName clusterName() {
-        return this.clusterName;
-    }
-
     public ClusterName getClusterName() {
-        return clusterName();
-    }
-
-    public DiscoveryNode[] nodes() {
-        return this.nodes;
+        return this.clusterName;
     }
 
     public DiscoveryNode[] getNodes() {
-        return nodes();
+        return this.nodes;
     }
 
     @Override

src/main/java/org/elasticsearch/rest/action/admin/cluster/node/shutdown/RestNodesShutdownAction.java

@@ -61,10 +61,10 @@ public void onResponse(NodesShutdownResponse response) {
                 try {
                     XContentBuilder builder = restContentBuilder(request);
                     builder.startObject();
-                    builder.field("cluster_name", response.clusterName().value());
+                    builder.field("cluster_name", response.getClusterName().value());
 
                     builder.startObject("nodes");
-                    for (DiscoveryNode node : response.nodes()) {
+                    for (DiscoveryNode node : response.getNodes()) {
                         builder.startObject(node.id(), XContentBuilder.FieldCaseConversion.NONE);
                         builder.field("name", node.name(), XContentBuilder.FieldCaseConversion.NONE);
                         builder.endObject();