ArafatKhan2198 commented on code in PR #4519:
URL: https://github.com/apache/ozone/pull/4519#discussion_r1185737268


##########
hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/ClusterStateEndpoint.java:
##########
@@ -136,7 +136,7 @@ public Response getClusterState() {
     }
 
     Long totalKeys = 0L;
-    Long deletedKeys = 0L;
+    Long keysAwaitingDeletion = 0L;

Review Comment:
   @smengcl, I completely agree with your point on consistency. After reviewing 
the code snippet you posted above, I believe it would be better to revert back 
to the previous name we used, 'keysPendingDeletion', since we are already using 
something similar in the ozone code.
   
   @smitajoshi12, I think we can revert back to the previous name!



-- 
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