You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by sn...@apache.org on 2024/03/18 22:24:54 UTC
(pinot) branch master updated: Logging a warn message instead of throwing exception (#12546)
This is an automated email from the ASF dual-hosted git repository.
snlee 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 def8a47f56 Logging a warn message instead of throwing exception (#12546)
def8a47f56 is described below
commit def8a47f5650c3910a99803e0add7abb2be5d56a
Author: swaminathanmanish <12...@users.noreply.github.com>
AuthorDate: Mon Mar 18 15:24:47 2024 -0700
Logging a warn message instead of throwing exception (#12546)
---
.../apache/pinot/controller/helix/core/minion/PinotTaskManager.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/PinotTaskManager.java b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/PinotTaskManager.java
index 97f246223a..d2086dfb1c 100644
--- a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/PinotTaskManager.java
+++ b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/PinotTaskManager.java
@@ -47,7 +47,6 @@ import org.apache.pinot.common.minion.TaskGeneratorMostRecentRunInfo;
import org.apache.pinot.common.minion.TaskManagerStatusCache;
import org.apache.pinot.controller.ControllerConf;
import org.apache.pinot.controller.LeadControllerManager;
-import org.apache.pinot.controller.api.exception.NoTaskScheduledException;
import org.apache.pinot.controller.api.exception.TaskAlreadyExistsException;
import org.apache.pinot.controller.api.exception.UnknownTaskTypeException;
import org.apache.pinot.controller.helix.core.PinotHelixResourceManager;
@@ -212,7 +211,7 @@ public class PinotTaskManager extends ControllerPeriodicTask<Void> {
taskGenerator.getMaxAttemptsPerTask()));
}
if (responseMap.isEmpty()) {
- throw new NoTaskScheduledException("No task scheduled for 'tableName': " + tableName);
+ LOGGER.warn("No task submitted for tableName: {}", tableName);
}
return responseMap;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org