ChenSammi commented on code in PR #7015:
URL: https://github.com/apache/ozone/pull/7015#discussion_r1699935426
##########
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java:
##########
@@ -2684,6 +2686,24 @@ public void setTimes(OzoneObj obj, String keyName, long
mtime, long atime)
ozoneManagerClient.setTimes(builder.build(), mtime, atime);
}
+ @Override
+ public LeaseKeyInfo recoverLease(String volumeName, String bucketName,
+ String keyName, boolean force)
+ throws IOException {
+ if (omVersion.compareTo(OzoneManagerVersion.HBASE_SUPPORT) < 0) {
+ throw new IOException("OzoneManager does not support lease recovery");
+ }
+ return ozoneManagerClient.recoverLease(volumeName, bucketName, keyName,
force);
+ }
+
+ @Override
+ public void recoverKey(OmKeyArgs args, long clientID) throws IOException {
+ if (omVersion.compareTo(OzoneManagerVersion.HBASE_SUPPORT) < 0) {
+ throw new IOException("OzoneManager does not support lease recovery");
Review Comment:
Same suggestion.
##########
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java:
##########
@@ -2684,6 +2686,24 @@ public void setTimes(OzoneObj obj, String keyName, long
mtime, long atime)
ozoneManagerClient.setTimes(builder.build(), mtime, atime);
}
+ @Override
+ public LeaseKeyInfo recoverLease(String volumeName, String bucketName,
+ String keyName, boolean force)
+ throws IOException {
+ if (omVersion.compareTo(OzoneManagerVersion.HBASE_SUPPORT) < 0) {
+ throw new IOException("OzoneManager does not support lease recovery");
+ }
+ return ozoneManagerClient.recoverLease(volumeName, bucketName, keyName,
force);
+ }
+
+ @Override
+ public void recoverKey(OmKeyArgs args, long clientID) throws IOException {
+ if (omVersion.compareTo(OzoneManagerVersion.HBASE_SUPPORT) < 0) {
+ throw new IOException("OzoneManager does not support lease recovery");
Review Comment:
Same suggestion as above.
--
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]
