Distinguish read and write request count in region

Details

Description

Distinguishing read and write request counts, on top of HBASE-3507, would benefit load balancer.
The action for balancing read vs. write load should be different. For read load, region movement should be low (to keep scanner happy). For write load, region movement is allowed.
Now that we have cheap(er) counters, it should not be too burdensome keeping up the extra count.

+1 on patch. Looks good Ted. Need to make sure though that we up the version on the Interface since the serialization of HSL has been changed by this patch (Unfortunately, HSL is not versioned so we can't have it migrate itself if it gets wrong version).

stack
added a comment - 24/Mar/11 23:32 +1 on patch. Looks good Ted. Need to make sure though that we up the version on the Interface since the serialization of HSL has been changed by this patch (Unfortunately, HSL is not versioned so we can't have it migrate itself if it gets wrong version).

To be clear, for this patch to be committed, we will need to up our rpc version on all proxy interfaces. HServerLoad is buried inside HServerInfo at the moment. Its looking like pressure is building such that we will have to up our version numbers – hbase-1502 where we rejigger heartbeat includes deprecation of HSA and redo of HSI – but also Gary has patch to include cluster id in ClusterStatue.... So, the RPC version bump looks like it'll happen soon. We'll apply this patch then.

stack
added a comment - 01/Apr/11 17:40 To be clear, for this patch to be committed, we will need to up our rpc version on all proxy interfaces. HServerLoad is buried inside HServerInfo at the moment. Its looking like pressure is building such that we will have to up our version numbers – hbase-1502 where we rejigger heartbeat includes deprecation of HSA and redo of HSI – but also Gary has patch to include cluster id in ClusterStatue.... So, the RPC version bump looks like it'll happen soon. We'll apply this patch then.

Committed to TRUNK. My dementia on versions above should be ignored. We've already upped our rpc version across the board because of CoProcessors change. This change goes in under that same version change. Thanks for the patch Ted and you did us a service Versioning the HSI and HSL classes.

stack
added a comment - 01/Apr/11 23:15 Committed to TRUNK. My dementia on versions above should be ignored. We've already upped our rpc version across the board because of CoProcessors change. This change goes in under that same version change. Thanks for the patch Ted and you did us a service Versioning the HSI and HSL classes.

This issue was closed as part of a bulk closing operation on 2015-11-20. All issues that have been resolved and where all fixVersions have been released have been closed (following discussions on the mailing list).

Lars Francke
added a comment - 20/Nov/15 12:43 This issue was closed as part of a bulk closing operation on 2015-11-20. All issues that have been resolved and where all fixVersions have been released have been closed (following discussions on the mailing list).