kaijchen commented on code in PR #3226:
URL: https://github.com/apache/ozone/pull/3226#discussion_r861441112
##########
hadoop-ozone/interface-storage/src/main/java/org/apache/hadoop/ozone/om/OMMetadataManager.java:
##########
@@ -260,13 +261,14 @@ List<OmVolumeArgs> listVolumes(String userName, String
prefix,
* Returns the names of up to {@code count} open keys whose age is
* greater than or equal to {@code expireThreshold}.
*
- * @param count The maximum number of open keys to return.
+ * @param limit The maximum number of open keys to return.
* @param expireThreshold The threshold of open key expire age.
- * @return a list of {@link String} representing names of open expired keys.
+ * @param bucketLayout The type of open keys to get (e.g. DEFAULT or FSO).
+ * @return a {@link List} of {@link OpenKeyBucket}, the expired open keys.
Review Comment:
Feels redundant to me, and it breaks line line limit.
--
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]
