From a2ff140f43a19eb0e9661ce6b1dbffe14fcdd11e Mon Sep 17 00:00:00 2001 From: Wenjun Ruan Date: Thu, 24 Nov 2022 21:26:52 +0800 Subject: [PATCH] Fix sql task will send alert if we don't choose the send email (#12984) --- .../org/apache/dolphinscheduler/plugin/task/sql/SqlTask.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dolphinscheduler-task-plugin/dolphinscheduler-task-sql/src/main/java/org/apache/dolphinscheduler/plugin/task/sql/SqlTask.java b/dolphinscheduler-task-plugin/dolphinscheduler-task-sql/src/main/java/org/apache/dolphinscheduler/plugin/task/sql/SqlTask.java index 391899dc5..7fe98c78f 100644 --- a/dolphinscheduler-task-plugin/dolphinscheduler-task-sql/src/main/java/org/apache/dolphinscheduler/plugin/task/sql/SqlTask.java +++ b/dolphinscheduler-task-plugin/dolphinscheduler-task-sql/src/main/java/org/apache/dolphinscheduler/plugin/task/sql/SqlTask.java @@ -319,7 +319,7 @@ public class SqlTask extends AbstractTask { String result = resultJSONArray.isEmpty() ? JSONUtils.toJsonString(generateEmptyRow(resultSet)) : JSONUtils.toJsonString(resultJSONArray); - if (sqlParameters.getSendEmail() == null || sqlParameters.getSendEmail()) { + if (Boolean.TRUE.equals(sqlParameters.getSendEmail())) { sendAttachment(sqlParameters.getGroupId(), StringUtils.isNotEmpty(sqlParameters.getTitle()) ? sqlParameters.getTitle() : taskExecutionContext.getTaskName() + " query result sets", result); -- GitLab