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

Jesse Yates commented on HBASE-7415:
------------------------------------

It might be cool to update the monitor with how many regions have reported for 
each step in the online case. But that might be a bit more work than is worth 
it.

One comment, looks like an extra whitespace crept in here:
{code}
@@ -66,6 +68,7 @@ public class DisabledTableSnapshotHandler extends 
TakeSnapshotHandler {
     super(snapshot, masterServices);
 
     // setup the timer
+
     timeoutInjector = 
TakeSnapshotUtils.getMasterTimerAndBindToMonitor(snapshot, conf, monitor);
   }
{code}

Otherwise, +1
                
> [snapshots] Add task information to snapshot operation
> ------------------------------------------------------
>
>                 Key: HBASE-7415
>                 URL: https://issues.apache.org/jira/browse/HBASE-7415
>             Project: HBase
>          Issue Type: New Feature
>          Components: Client, master, regionserver, snapshots, Zookeeper
>            Reporter: Jesse Yates
>            Assignee: Jesse Yates
>             Fix For: 0.95.0
>
>         Attachments: hbase-7415-v0.patch, hbase-7415-v1.patch, 
> HBASE-7415-v1-rebase.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