fapifta commented on code in PR #7060:
URL: https://github.com/apache/ozone/pull/7060#discussion_r1730996066
##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/OmUtils.java:
##########
@@ -243,6 +243,10 @@ public static boolean isReadOnly(
case ListKeys:
case ListKeysLight:
case ListTrash:
+ // This is deprecated by HDDS-11251. Keeping this in here
Review Comment:
For me it is not clear if this comment is talking about ListTrash, or
ServiceList. Can we write ListTrash is deprecated... instead? I know if one
gets the git annotate output, or if one checks the JIRA then it is clear what
the comment is talking about, but that requires an extra hop, so I would like
to make our life easier later with this change. Same goes to the other comment
for RecoverTrash.
--
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]
