sarvekshayr opened a new pull request, #7670: URL: https://github.com/apache/ozone/pull/7670
## What changes were proposed in this pull request? The `ozone debug recover` command that is currently used for lease recovery has a few problems: - It modifies the cluster, so it should not be under `debug`. - The name does not specify that it deals with leases. - It uses a standard API from the `FileSystem` interface unlike other low level tools in this package. The command is unique because it goes through the `FileSystem` interface but is not exposed through the `fs` command line in Hadoop or Ozone. For these reasons, we propose moving this command to its own Ozone subcommand: `ozone admin om lease recover`. This also allows other lease related subcommands (like querying of existing leases) to be added in the future. ## What is the link to the Apache JIRA [HDDS-12029](https://issues.apache.org/jira/browse/HDDS-12029) ## How was this patch tested? Tested the patch on a docker cluster. ``` bash-5.1$ ozone admin om lease Missing required subcommand Usage: ozone admin om lease [COMMAND] Command for all lease related queries. Commands: recover Recover the lease of a specified file. Make sure to specify file system scheme if ofs:// is not the default. ``` ``` bash-5.1$ ozone admin om lease recover --path=/vol1/bucket1/key1 Lease recovery SUCCEEDED on /vol1/bucket1/key1 ``` -- 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]
