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

Ted Yu commented on HBASE-7615:
-------------------------------

Went through the patch which looks good.
{code}
+        new CloneSnapshotHandler(master, snapshot, hTableDescriptor, 
metricsMaster).prepare();
{code}
nit: both metricsMaster and this.metricsMaster are used. It would be nice to be 
consistent.

Hadoop QA is down again - may need to run test suite manually or later.
                
> Add metrics for snapshots
> -------------------------
>
>                 Key: HBASE-7615
>                 URL: https://issues.apache.org/jira/browse/HBASE-7615
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Ted Yu
>            Assignee: Matteo Bertozzi
>         Attachments: HBASE-7615-v0.patch, HBASE-7615-v0-ui-corrupted.png, 
> HBASE-7615-v0-ui.png, HBASE-7615-v1.patch
>
>
> Metrics should be added for snapshot.
> From Matteo: Output that we have in SnapshotInfo should be covered:
> "%d HFiles (%d in archive), total size %s (%.2f%% %s shared with the source 
> table)"
> Jesse mentioned snaphot counts, average time to completion.
> I think we should have counts for successful / failed snapshots.

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