From 346c9786165c5d67a780f5ce010e5da76b5ac129 Mon Sep 17 00:00:00 2001 From: fengyu Date: Tue, 12 Jan 2016 12:04:45 +0800 Subject: [PATCH] admin email list can only get the first one Signed-off-by: fengyu --- common/src/main/java/org/apache/kylin/common/KylinConfig.java | 4 ++-- job/src/main/java/org/apache/kylin/job/engine/JobEngineConfig.java | 2 +- .../main/java/org/apache/kylin/job/execution/AbstractExecutable.java | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/common/src/main/java/org/apache/kylin/common/KylinConfig.java b/common/src/main/java/org/apache/kylin/common/KylinConfig.java index 25800d8..b7cc8a2 100644 --- a/common/src/main/java/org/apache/kylin/common/KylinConfig.java +++ b/common/src/main/java/org/apache/kylin/common/KylinConfig.java @@ -426,8 +426,8 @@ public class KylinConfig { return getOptionalStringArray(KYLIN_REST_SERVERS); } - public String getAdminDls() { - return getOptional("kylin.job.admin.dls", null); + public String[] getAdminDls() { + return getOptionalStringArray("kylin.job.admin.dls"); } public long getJobStepTimeout() { diff --git a/job/src/main/java/org/apache/kylin/job/engine/JobEngineConfig.java b/job/src/main/java/org/apache/kylin/job/engine/JobEngineConfig.java index 6b005be..a063697 100644 --- a/job/src/main/java/org/apache/kylin/job/engine/JobEngineConfig.java +++ b/job/src/main/java/org/apache/kylin/job/engine/JobEngineConfig.java @@ -129,7 +129,7 @@ public class JobEngineConfig { /** * @return the adminDls */ - public String getAdminDls() { + public String[] getAdminDls() { return config.getAdminDls(); } diff --git a/job/src/main/java/org/apache/kylin/job/execution/AbstractExecutable.java b/job/src/main/java/org/apache/kylin/job/execution/AbstractExecutable.java index 8eea1cf..a49cbd3 100644 --- a/job/src/main/java/org/apache/kylin/job/execution/AbstractExecutable.java +++ b/job/src/main/java/org/apache/kylin/job/execution/AbstractExecutable.java @@ -199,9 +199,9 @@ public abstract class AbstractExecutable implements Executable, Idempotent { try { List users = Lists.newArrayList(); users.addAll(getNotifyList()); - final String adminDls = KylinConfig.getInstanceFromEnv().getAdminDls(); + final String[] adminDls = KylinConfig.getInstanceFromEnv().getAdminDls(); if (null != adminDls) { - for (String adminDl : adminDls.split(",")) { + for (String adminDl : adminDls) { users.add(adminDl); } } -- 1.9.4.msysgit.2