dependent node log output optimization and fix checkstyle (#10593)
(cherry picked from commit 397e51fa63)
This commit is contained in:
parent
1e498a5f89
commit
b0113a9b64
|
|
@ -17,13 +17,12 @@
|
|||
|
||||
package org.apache.dolphinscheduler.server.master.runner.task;
|
||||
|
||||
import static org.apache.dolphinscheduler.common.Constants.DEPENDENT_SPLIT;
|
||||
import static org.apache.dolphinscheduler.plugin.task.api.TaskConstants.TASK_TYPE_DEPENDENT;
|
||||
|
||||
import org.apache.dolphinscheduler.plugin.task.api.enums.TaskTimeoutStrategy;
|
||||
import org.apache.dolphinscheduler.common.utils.NetUtils;
|
||||
import org.apache.dolphinscheduler.plugin.task.api.enums.DependResult;
|
||||
import org.apache.dolphinscheduler.plugin.task.api.enums.ExecutionStatus;
|
||||
import org.apache.dolphinscheduler.plugin.task.api.enums.TaskTimeoutStrategy;
|
||||
import org.apache.dolphinscheduler.plugin.task.api.model.DependentTaskModel;
|
||||
import org.apache.dolphinscheduler.plugin.task.api.parameters.DependentParameters;
|
||||
import org.apache.dolphinscheduler.plugin.task.api.utils.DependentUtils;
|
||||
|
|
@ -111,8 +110,8 @@ public class DependentTaskProcessor extends BaseTaskProcessor {
|
|||
&& TaskTimeoutStrategy.WARNFAILED != taskTimeoutStrategy) {
|
||||
return true;
|
||||
}
|
||||
logger.info("dependent task {} timeout, strategy {} ",
|
||||
taskInstance.getId(), taskTimeoutStrategy.getDescp());
|
||||
logger.info("dependent taskInstanceId: {} timeout, taskName: {}, strategy: {} ",
|
||||
taskInstance.getId(), taskInstance.getName(), taskTimeoutStrategy.getDescp());
|
||||
result = DependResult.FAILED;
|
||||
endTask();
|
||||
return true;
|
||||
|
|
@ -161,7 +160,7 @@ public class DependentTaskProcessor extends BaseTaskProcessor {
|
|||
if (!dependResultMap.containsKey(entry.getKey())) {
|
||||
dependResultMap.put(entry.getKey(), entry.getValue());
|
||||
//save depend result to log
|
||||
logger.info("dependent item complete {} {},{}", DEPENDENT_SPLIT, entry.getKey(), entry.getValue());
|
||||
logger.info("dependent item complete, task: {}, result: {}", entry.getKey(), entry.getValue());
|
||||
}
|
||||
}
|
||||
if (!dependentExecute.finish(dependentDate)) {
|
||||
|
|
@ -183,7 +182,7 @@ public class DependentTaskProcessor extends BaseTaskProcessor {
|
|||
dependResultList.add(dependResult);
|
||||
}
|
||||
result = DependentUtils.getDependResultForRelation(this.dependentParameters.getRelation(), dependResultList);
|
||||
logger.info("dependent task completed, dependent result:{}", result);
|
||||
logger.info("dependent task completed, dependent result: {}", result);
|
||||
return result;
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue