[
https://issues.apache.org/jira/browse/HBASE-20630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504026#comment-16504026
]
Ted Yu commented on HBASE-20630:
--------------------------------
{code}
+ private void executeDeleteOlderThan(CommandLine cmdline) throws
IOException {
+ String value = cmdline.getOptionValue(OPTION_PATH);
{code}
The deletion operation shouldn't depend on Path option, right ?
It seems you use Path option for deletion. I think you meant to use Keep option.
{code}
import static
org.apache.hadoop.hbase.backup.BackupRestoreConstants.OPTION_PATH_DESC;
+import static
org.apache.hadoop.hbase.backup.BackupRestoreConstants.OPTION_PATH;
{code}
Order should be reversed.
> B&R: Delete command enhancements
> --------------------------------
>
> Key: HBASE-20630
> URL: https://issues.apache.org/jira/browse/HBASE-20630
> Project: HBase
> Issue Type: New Feature
> Reporter: Vladimir Rodionov
> Assignee: Vladimir Rodionov
> Priority: Major
> Attachments: HBASE-20630-v1.patch, HBASE-20630-v2.patch,
> HBASE-20630-v3.patch
>
>
> Make the command more useable. Currently, user needs to provide list of
> backup ids to delete. It would be nice to have more convenient options, such
> as: deleting all backups which are older than XXX days, etc
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
