This is an automated email from the ASF dual-hosted git repository.

Jackie-Jiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 772f7723380 [test] Stabilize table rebalance disk utilization 
prechecks (#20135)
772f7723380 is described below

commit 772f77233801a8f256b4a045ade4cf46e1ce7e8d
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Mon Nov 2 21:06:56 2026 -0800

    [test] Stabilize table rebalance disk utilization prechecks (#20135)
---
 .../pinot/integration/tests/TableRebalanceIntegrationTest.java     | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/TableRebalanceIntegrationTest.java
 
b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/TableRebalanceIntegrationTest.java
index ed24dcaf23f..10a38411a33 100644
--- 
a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/TableRebalanceIntegrationTest.java
+++ 
b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/TableRebalanceIntegrationTest.java
@@ -74,6 +74,8 @@ public class TableRebalanceIntegrationTest extends 
BaseHybridClusterIntegrationT
   protected void overrideControllerConf(Map<String, Object> properties) {
     super.overrideControllerConf(properties);
     properties.put(ControllerConf.CONFIG_OF_MAX_TABLE_REBALANCE_JOBS_IN_ZK, 2);
+    // Keep disk usage unavailable for the rebalance pre-checks, regardless of 
how long the test takes.
+    properties.put(ControllerConf.RESOURCE_UTILIZATION_CHECKER_FREQUENCY, 0);
   }
 
   @Test
@@ -879,10 +881,7 @@ public class TableRebalanceIntegrationTest extends 
BaseHybridClusterIntegrationT
         expectedReplicaGroupStatus);
     
assertEquals(preChecksResult.get(DefaultRebalancePreChecker.REPLICA_GROUPS_INFO).getMessage(),
         expectedReplicaGroupMessage);
-    // As the disk utilization check periodic task was disabled in the test 
controller (ControllerConf
-    // .RESOURCE_UTILIZATION_CHECKER_INITIAL_DELAY was set to 30000s, see 
org.apache.pinot.controller.helix
-    // .ControllerTest.getDefaultControllerConfiguration), server's disk util 
should be unavailable on all servers if
-    // not explicitly set via 
org.apache.pinot.controller.validation.ResourceUtilizationInfo.setDiskUsageInfo
+    // Periodic resource utilization collection is disabled in 
overrideControllerConf(), so disk usage is unavailable.
     
assertEquals(preChecksResult.get(DefaultRebalancePreChecker.DISK_UTILIZATION).getPreCheckStatus(),
         RebalancePreCheckerResult.PreCheckStatus.WARN);
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to