avijayanhwx commented on a change in pull request #1405:
URL: https://github.com/apache/hadoop-ozone/pull/1405#discussion_r485769011



##########
File path: 
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/OMBucketSetPropertyRequest.java
##########
@@ -206,4 +207,8 @@ public OMClientResponse validateAndUpdateCache(OzoneManager 
ozoneManager,
       return omClientResponse;
     }
   }
+
+  public static String getRequestType() {

Review comment:
       @bharatviswa504 Yes, I pondered about adding annotation for this. Given 
that we already have annotations for Cleanup tables and BelongsToLayoutVersion, 
I wanted to hold off on adding more annotations. If it is ok with you, after 
the first round of changes, we can change this to an annotation.




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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org

Reply via email to