[ 
https://issues.apache.org/jira/browse/HBASE-20631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504928#comment-16504928
 ] 

Ted Yu commented on HBASE-20631:
--------------------------------

{code}
+  String OPTION_MERGE_DAYS_DESC = "Specifies number of last days to merge 
incremental backups";
{code}
Can you make the description more readable ? What does 'last days' mean ?
{code}
+        throw new IOException(INCORRECT_USAGE);
{code}
It would be better saying what is missing w.r.t. the merge command.
{code}
     } catch (Exception ee) {
+      ee.printStackTrace();
{code}
Drop the stack trace.



> B&R: Merge command enhancements 
> --------------------------------
>
>                 Key: HBASE-20631
>                 URL: https://issues.apache.org/jira/browse/HBASE-20631
>             Project: HBase
>          Issue Type: New Feature
>            Reporter: Vladimir Rodionov
>            Assignee: Vladimir Rodionov
>            Priority: Major
>         Attachments: HBASE-20631-v1.patch
>
>
> Currently, merge supports only list of backup ids, which users must provide. 
> Date range merges seem more convenient for users. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to