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

Andrew Purtell commented on HBASE-11516:
----------------------------------------

I like the idea but not the proposed changes.

Suggestions

1) Expose this as a metric instead of changes to ClusterStatus. 

or

2) When changing ClusterStatus make it easy to add more metrics like this over 
time, something like:

{noformat}
repeated Metric metrics = 16;

message Metric {
    required string name = 1;
    required uint64 value = 2;
}
{noformat}

I'd prefer option 1. I don't think we should be passing metrics through 
ClusterStatus. However, if consensus opinion is we can and should do that here, 
then I prefer option 2 over the current patch.

> Track time spent in executing coprocessors in each region.
> ----------------------------------------------------------
>
>                 Key: HBASE-11516
>                 URL: https://issues.apache.org/jira/browse/HBASE-11516
>             Project: HBase
>          Issue Type: Improvement
>          Components: Coprocessors
>    Affects Versions: 0.98.4
>            Reporter: Srikanth Srungarapu
>            Assignee: Srikanth Srungarapu
>            Priority: Minor
>             Fix For: 0.98.5
>
>         Attachments: HBASE-11516.patch
>
>
> Currently, the time spent in executing coprocessors is not yet being tracked. 
> This feature can be handy for debugging coprocessors in case of any trouble.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to