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

Matteo Bertozzi commented on HBASE-8642:
----------------------------------------

overall looks good to me...
Inside deleteSnapshotsByTable() you can use the SnapshotDescription of the for 
loop, instead of building a new one.
Do you want also add the shell support?
                
> [Snapshot] List and delete snapshot by table
> --------------------------------------------
>
>                 Key: HBASE-8642
>                 URL: https://issues.apache.org/jira/browse/HBASE-8642
>             Project: HBase
>          Issue Type: Improvement
>          Components: snapshots
>    Affects Versions: 0.98.0, 0.95.0, 0.95.1, 0.95.2
>            Reporter: Julian Zhou
>            Assignee: Julian Zhou
>            Priority: Minor
>             Fix For: 0.98.0, 0.95.0, 0.95.1, 0.95.2
>
>         Attachments: 8642-trunk-0.95-v0.patch
>
>
> Support list and delete snapshot by table name.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to