sreejasahithi commented on code in PR #8523:
URL: https://github.com/apache/ozone/pull/8523#discussion_r2246845454


##########
hadoop-ozone/cli-admin/src/main/java/org/apache/hadoop/hdds/scm/cli/datanode/BasicDatanodeInfo.java:
##########
@@ -19,84 +19,70 @@
 
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.annotation.JsonInclude;
+import com.fasterxml.jackson.annotation.JsonPropertyOrder;
 import java.util.List;
 import org.apache.hadoop.hdds.protocol.DatanodeDetails;
 import org.apache.hadoop.hdds.protocol.proto.HddsProtos;
 
 /**
  * Represents filtered Datanode information for json use.
  */
-
-public class BasicDatanodeInfoJson {
-  private final String id;
-  private final String hostName;
-  private final String ipAddress;
-  private final List<DatanodeDetails.Port> ports;
-  private final long setupTime;
-  private final int currentVersion;
-  private final HddsProtos.NodeOperationalState persistedOpState;
-  private final HddsProtos.NodeOperationalState opState;
-  private final long persistedOpStateExpiryEpochSec;
-  private final HddsProtos.NodeState healthState;
-  private final boolean decommissioned;
-  private final boolean maintenance;
-  private final int level;
-  private final int cost;
-  private final int numOfLeaves;
-  private final String networkFullPath;
-  private final String networkLocation;
-  private final String networkName;
+@JsonPropertyOrder({

Review Comment:
   Okay, this is better, will Update it.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to