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

Ted Yu commented on HBASE-16198:
--------------------------------

{code}
415             return TableName.valueOf(value);
416           } catch (Exception e){
{code}
Catching IllegalArgumentException should be enough.

For BackupAdmin.java :
{code}
111        * @param naem - table's name
{code}
naem: typo
{code}
146           List<BackupInfo> history = table.getBackupHistory(); 
{code}
Can variant of getBackupHistory() be added which takes TableName as parameter ?
This would limit the amount of data retrieved.

> Enhance backup history command
> ------------------------------
>
>                 Key: HBASE-16198
>                 URL: https://issues.apache.org/jira/browse/HBASE-16198
>             Project: HBase
>          Issue Type: New Feature
>    Affects Versions: 2.0.0
>            Reporter: Vladimir Rodionov
>            Assignee: Vladimir Rodionov
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16198-v1.patch
>
>
> We need history per table.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to