bsglz commented on a change in pull request #1926:
URL: https://github.com/apache/hbase/pull/1926#discussion_r447376937
##########
File path: hbase-protocol-shaded/src/main/protobuf/server/ClusterStatus.proto
##########
@@ -133,8 +133,8 @@ message RegionLoad {
/** the most recent sequence Id from cache flush */
optional uint64 complete_sequence_id = 15;
- /** The current data locality for region in the regionserver */
- optional float data_locality = 16;
+ /** The current blocks local weight for region in the regionserver */
+ optional uint64 blocks_local_weight = 16;
Review comment:
"Can we just add float data_locality_for_ssd at the end"
You mean blocks_local_weight?
I had thought about it, just lead to some redundancy, but you are right that
the compatibility should not be breaked in minor, will fix later.
Thanks.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
