HDFS-16326. Simplify the code for DiskBalancer (#3663)
Reviewed-by: Viraj Jasani <vjasani@apache.org>
This commit is contained in:
parent
0c1eb0125b
commit
965e3874e9
|
@ -808,7 +808,7 @@ public class DiskBalancer {
|
||||||
long bytesToCopy = item.getBytesToCopy() - item.getBytesCopied();
|
long bytesToCopy = item.getBytesToCopy() - item.getBytesCopied();
|
||||||
bytesToCopy = bytesToCopy +
|
bytesToCopy = bytesToCopy +
|
||||||
((bytesToCopy * getBlockTolerancePercentage(item)) / 100);
|
((bytesToCopy * getBlockTolerancePercentage(item)) / 100);
|
||||||
return (blockSize <= bytesToCopy) ? true : false;
|
return blockSize <= bytesToCopy;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -833,7 +833,7 @@ public class DiskBalancer {
|
||||||
private boolean isCloseEnough(DiskBalancerWorkItem item) {
|
private boolean isCloseEnough(DiskBalancerWorkItem item) {
|
||||||
long temp = item.getBytesCopied() +
|
long temp = item.getBytesCopied() +
|
||||||
((item.getBytesCopied() * getBlockTolerancePercentage(item)) / 100);
|
((item.getBytesCopied() * getBlockTolerancePercentage(item)) / 100);
|
||||||
return (item.getBytesToCopy() >= temp) ? false : true;
|
return item.getBytesToCopy() < temp;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in New Issue