You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by sh...@apache.org on 2020/03/02 01:40:08 UTC
[kylin] branch 3.0.x updated: KYLIN-4132 Kylin needn't use
"org.apache.directory.api.util.Strings" to import extra dependency
This is an automated email from the ASF dual-hosted git repository.
shaofengshi pushed a commit to branch 3.0.x
in repository https://gitbox.apache.org/repos/asf/kylin.git
The following commit(s) were added to refs/heads/3.0.x by this push:
new 9a86c37 KYLIN-4132 Kylin needn't use "org.apache.directory.api.util.Strings" to import extra dependency
9a86c37 is described below
commit 9a86c3731f800bd9b2ac70234249f462f8016fc8
Author: rupengwang <wa...@live.cn>
AuthorDate: Sun Mar 1 00:27:16 2020 +0800
KYLIN-4132 Kylin needn't use "org.apache.directory.api.util.Strings" to import extra dependency
---
.../src/main/java/org/apache/kylin/rest/service/JobService.java | 9 ++++-----
.../main/java/org/apache/kylin/rest/service/ProjectService.java | 4 ++--
2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/JobService.java b/server-base/src/main/java/org/apache/kylin/rest/service/JobService.java
index 9074a54..f0dc3c6 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/service/JobService.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/service/JobService.java
@@ -34,7 +34,6 @@ import java.util.TimeZone;
import javax.annotation.Nullable;
import org.apache.commons.lang3.StringUtils;
-import org.apache.directory.api.util.Strings;
import org.apache.kylin.common.KylinConfig;
import org.apache.kylin.common.lock.DistributedLock;
import org.apache.kylin.common.util.Pair;
@@ -902,7 +901,7 @@ public class JobService extends BasicService implements InitializingBean {
return false;
}
- if (Strings.isEmpty(jobName)) {
+ if (StringUtils.isEmpty(jobName)) {
return true;
}
@@ -1002,7 +1001,7 @@ public class JobService extends BasicService implements InitializingBean {
return false;
}
- if (Strings.isEmpty(jobName)) {
+ if (StringUtils.isEmpty(jobName)) {
return true;
}
@@ -1226,7 +1225,7 @@ public class JobService extends BasicService implements InitializingBean {
return false;
}
- if (Strings.isEmpty(jobName)) {
+ if (StringUtils.isEmpty(jobName)) {
return true;
}
@@ -1301,7 +1300,7 @@ public class JobService extends BasicService implements InitializingBean {
return false;
}
- if (Strings.isEmpty(jobName)) {
+ if (StringUtils.isEmpty(jobName)) {
return true;
}
diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java b/server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java
index 2677578..c1c4612 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java
@@ -29,7 +29,7 @@ import java.util.Set;
import javax.annotation.Nullable;
import com.google.common.collect.Sets;
-import org.apache.directory.api.util.Strings;
+import org.apache.commons.lang3.StringUtils;
import org.apache.kylin.metadata.project.ProjectInstance;
import org.apache.kylin.metadata.realization.RealizationType;
import org.apache.kylin.rest.constant.Constant;
@@ -186,7 +186,7 @@ public class ProjectService extends BasicService {
}
// listAll method may not need a single param.But almost all listAll method pass
- if (!Strings.isEmpty(projectName)) {
+ if (!StringUtils.isEmpty(projectName)) {
readableProjects = Lists
.newArrayList(Iterators.filter(readableProjects.iterator(), new Predicate<ProjectInstance>() {
@Override