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 2016/11/02 08:44:24 UTC
[11/33] kylin git commit: Revert "KYLIN-2137 add check for
'kylin.job.mr.lib.dir'"
Revert "KYLIN-2137 add check for 'kylin.job.mr.lib.dir'"
This reverts commit a37ac46ace9fceda9780121d97348d6eb620f008.
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/11c2c691
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/11c2c691
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/11c2c691
Branch: refs/heads/v1.6.0-rc1-cdh5.7
Commit: 11c2c6919e7d2362c9aef5dca823752651615d57
Parents: a37ac46
Author: Li Yang <li...@apache.org>
Authored: Sat Oct 29 06:53:23 2016 +0800
Committer: Li Yang <li...@apache.org>
Committed: Sat Oct 29 06:53:23 2016 +0800
----------------------------------------------------------------------
.../apache/kylin/engine/mr/common/AbstractHadoopJob.java | 9 +--------
1 file changed, 1 insertion(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/11c2c691/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/AbstractHadoopJob.java
----------------------------------------------------------------------
diff --git a/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/AbstractHadoopJob.java b/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/AbstractHadoopJob.java
index a97a8c7..417d63d 100644
--- a/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/AbstractHadoopJob.java
+++ b/engine-mr/src/main/java/org/apache/kylin/engine/mr/common/AbstractHadoopJob.java
@@ -245,14 +245,7 @@ public abstract class AbstractHadoopJob extends Configured implements Tool {
// for KylinJobMRLibDir
String mrLibDir = kylinConf.getKylinJobMRLibDir();
- if (!StringUtils.isBlank(mrLibDir)) {
- File dirFileMRLIB = new File(mrLibDir);
- if (dirFileMRLIB.exists()) {
- StringUtil.appendWithSeparator(kylinDependency, mrLibDir);
- } else {
- logger.info("The directory '" + mrLibDir + "' for 'kylin.job.mr.lib.dir' does not exist!!!");
- }
- }
+ StringUtil.appendWithSeparator(kylinDependency, mrLibDir);
setJobTmpJarsAndFiles(job, kylinDependency.toString());