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

Abhishek Singh Chouhan commented on HBASE-18374:
------------------------------------------------

> The async client calls the mutate() for a single put. (see 
> RawAsyncTableImpl#put)
Having separate metrics for batch vs normal puts would be better then. Let me 
put up a patch that separates these. Thanks [~chia7712]

> The ambiguous is due to HTable#put. Perhaps HTable#put should use the 
> mutate() rather than BufferedMutator. There is a connected issue HBASE-17368.
I agree. I saw that HTable#put was using the multi route and the mutate put 
route was essentially unused, so thought that existing behavior should be 
sufficient.

> RegionServer Metrics improvements
> ---------------------------------
>
>                 Key: HBASE-18374
>                 URL: https://issues.apache.org/jira/browse/HBASE-18374
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 3.0.0
>            Reporter: Abhishek Singh Chouhan
>            Assignee: Abhishek Singh Chouhan
>             Fix For: 3.0.0
>
>         Attachments: HBASE-18374.branch-1.001.patch, 
> HBASE-18374.branch-1.001.patch, HBASE-18374.branch-1.002.patch, 
> HBASE-18374.master.001.patch, HBASE-18374.master.002.patch, 
> HBASE-18374.master.003.patch, HBASE-18374.master.004.patch
>
>
> At the RS level we have latency metrics for mutate/puts and deletes that are 
> updated per batch (ie. at the end of entire batchop if it contains put/delete 
> update the respective metric) in contrast with append/increment/get metrics 
> that are updated per op. This is a bit ambiguous since the delete and put 
> metrics are updated for multi row mutations that happen to contain a 
> put/delete. We should rename the metric(eg. delete_batch)/add better 
> description. Also we should add metrics for single delete client operations 
> that come through RSRpcServer.mutate path. We should also add metrics for 
> checkAndPut and checkAndDelete.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to