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

Matteo Bertozzi commented on HBASE-7415:
----------------------------------------

+1, nice stuff

just a minor nit: the "Restore x/y regions" is not really useful since a 
restore is composed by: regions to restore (this one below), regions to remove 
and regions to add. 
{code}
-    for (HRegionInfo hri: regions) restoreRegion(hri);
+    for (int i = 0; i < regions.size(); i++) {
+      HRegionInfo hri = regions.get(i);
+      restoreRegion(hri);
+      status.setStatus("Restored " + i + "/" + regions.size() + " regions");
+    }
{code}
                
> [snapshots] Add task information to snapshot operation
> ------------------------------------------------------
>
>                 Key: HBASE-7415
>                 URL: https://issues.apache.org/jira/browse/HBASE-7415
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Client, master, regionserver, snapshots, Zookeeper
>            Reporter: Jesse Yates
>            Assignee: Jesse Yates
>             Fix For: hbase-6055, 0.96.0
>
>         Attachments: hbase-7415-v0.patch, HBase_Snapshot_Task_UI.png
>
>
> Snapshot operations should have some sort of progresss information available 
> via the WebUI so admins can track progress of operations. This should go a 
> long way to enable 'good' admins to not hose their clusters by running 
> concurrent snapshot operations (e.g. rename while a clone is in progress).

--
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