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 f799d9075db Surface recorded scheduling errors when
assertNoTaskSchedule fails on null task names (#19389)
f799d9075db is described below
commit f799d9075db60884b0234ad998e79ed76380547f
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Fri Aug 28 16:25:22 2026 -0700
Surface recorded scheduling errors when assertNoTaskSchedule fails on null
task names (#19389)
---
.../java/org/apache/pinot/integration/tests/MinionTaskTestUtils.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/MinionTaskTestUtils.java
b/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/MinionTaskTestUtils.java
index 2291bbb8582..dbe60f75a8e 100644
---
a/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/MinionTaskTestUtils.java
+++
b/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/MinionTaskTestUtils.java
@@ -37,7 +37,9 @@ public class MinionTaskTestUtils {
}
public static void assertNoTaskSchedule(TaskSchedulingInfo info) {
- assertNotNull(info.getScheduledTaskNames());
+ // Null task names mean scheduling failed; surface the recorded errors
instead of a bare null assertion.
+ assertNotNull(info.getScheduledTaskNames(), "Scheduling failed with
generation errors: "
+ + info.getGenerationErrors() + ", scheduling errors: " +
info.getSchedulingErrors());
assertTrue(info.getScheduledTaskNames().isEmpty());
assertNoTaskErrors(info);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
