Skip to content

Commit

Permalink
Logging a warn message instead of throwing exception (apache#12546)
Browse files Browse the repository at this point in the history
  • Loading branch information
swaminathanmanish authored Mar 18, 2024
1 parent 6f54aac commit def8a47
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@
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;
Expand Down Expand Up @@ -212,7 +211,7 @@ public Map<String, String> createTask(String taskType, String tableName, @Nullab
taskGenerator.getMaxAttemptsPerTask()));
}
if (responseMap.isEmpty()) {
throw new NoTaskScheduledException("No task scheduled for 'tableName': " + tableName);
LOGGER.warn("No task submitted for tableName: {}", tableName);
}
return responseMap;
}
Expand Down

0 comments on commit def8a47

Please sign in to comment.