Gargi-jais11 commented on code in PR #9465:
URL: https://github.com/apache/ozone/pull/9465#discussion_r2621895632
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/diskbalancer/DiskBalancerService.java:
##########
@@ -639,9 +639,11 @@ public DiskBalancerInfo getDiskBalancerInfo() {
volumeDatadensity =
DiskBalancerVolumeCalculation.calculateVolumeDataDensity(immutableVolumeSet,
deltaSizes);
long bytesToMove = 0;
- if (this.operationalState == DiskBalancerRunningStatus.RUNNING) {
+ if (this.operationalState == DiskBalancerRunningStatus.RUNNING &&
+ !inProgressContainers.isEmpty()) {
Review Comment:
Yaa correct @ChenSammi thanks for bringing up this point. Its better to add
a comment in CLI output of status:
```
Note: EstimatedBytesToMove represents the total excess data that needs to be
moved to balance volumes, calculated based on volume utilization and threshold.
It may be non-zero even when no containers are currently being moved,
especially if the configured threshold is very small and none of the containers
meet the minimum size requirement for moving.
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
