xiaoyuyao commented on a change in pull request #329: HDDS-2423. Add the recover-trash command client side code URL: https://github.com/apache/hadoop-ozone/pull/329#discussion_r356411104
########## File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java ########## @@ -2238,6 +2238,15 @@ public void deleteKey(OmKeyArgs args) throws IOException { } } + // TODO: HDDS-2424. recover-trash command server side handling. + @Override + public boolean recoverTrash(String volumeName, String bucketName, + String keyName, String destinationBucket) throws IOException { + + boolean recoverOperation = true; + return recoverOperation; Review comment: Even this is a partial work, is IT possible to add a unit test for it, which can be easily extended later for HDDS-2424. ---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org