smengcl commented on a change in pull request #2857:
URL: https://github.com/apache/ozone/pull/2857#discussion_r755624379



##########
File path: 
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/OzoneVolume.java
##########
@@ -444,6 +471,14 @@ public void deleteBucket(String bucketName) throws 
IOException {
     proxy.deleteBucket(name, bucketName);
   }
 
+  public long getRefCount() {

Review comment:
       Initially I added this because findbugs was complaining that refCount is 
unused.
   
   I believe we could expose also refCount in `ozone volume info` / `ozone 
volume update`

##########
File path: 
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/OzoneVolume.java
##########
@@ -444,6 +471,14 @@ public void deleteBucket(String bucketName) throws 
IOException {
     proxy.deleteBucket(name, bucketName);
   }
 
+  public long getRefCount() {

Review comment:
       Initially I added this because findbugs was complaining that refCount is 
unused.
   
   I believe we could expose refCount in `ozone volume info` / `ozone volume 
update` as well.




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