bsglz commented on a change in pull request #1337:
URL: https://github.com/apache/hbase/pull/1337#discussion_r439795320
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/HDFSBlocksDistribution.java
##########
@@ -199,10 +225,23 @@ public long getUniqueBlocksTotalWeight() {
* @return the locality index of the given host
*/
public float getBlockLocalityIndex(String host) {
+ return getBlockLocalityIndex(host, false);
+ }
+ /**
+ * return the locality index of a given host
+ * @param host the host name
+ * @param forSsd only calc ssd type
+ * @return the locality index of the given host
+ */
+ public float getBlockLocalityIndex(String host, boolean forSsd) {
Review comment:
Yeah, thought about it, but i think we wont have such chance for a long
time, maybe rename is better.
----------------------------------------------------------------
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]
