You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by mc...@apache.org on 2021/06/28 17:09:51 UTC

[incubator-pinot] 01/01: Fixed a comment

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

mcvsubbu pushed a commit to branch minion-improvements
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit a9f9e8aab23f5ff9ecbda91b9db5d723e2c6d912
Author: Subbu Subramaniam <ss...@linkedin.com>
AuthorDate: Mon Jun 28 10:05:15 2021 -0700

    Fixed a comment
---
 .../pinot/integration/tests/SimpleMinionClusterIntegrationTest.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SimpleMinionClusterIntegrationTest.java b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SimpleMinionClusterIntegrationTest.java
index 930a41e..bcdf6a5 100644
--- a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SimpleMinionClusterIntegrationTest.java
+++ b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SimpleMinionClusterIntegrationTest.java
@@ -121,8 +121,8 @@ public class SimpleMinionClusterIntegrationTest extends ClusterTest {
     verifyTaskCount(task2, 0, 2, 0, 2);
 
     // Should not generate more tasks since SimpleMinionClusterIntegrationTests.NUM_TASKS is 2.
-    // Our test task generator does not generate if there are already this many sub-tasks in the
-    // running+waiting count already.
+    // Our test task generator does not generate if there are already this many top-level tasks
+    // in progress.
     assertNull(_taskManager.scheduleTasks().get(TASK_TYPE));
     assertNull(_taskManager.scheduleTask(TASK_TYPE));
 

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org