You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/08/18 08:35:40 UTC

kylin git commit: minor, remove dup dependency from tool

Repository: kylin
Updated Branches:
  refs/heads/master f77e032d0 -> 9629fa8fb


minor, remove dup dependency from tool


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/9629fa8f
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/9629fa8f
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/9629fa8f

Branch: refs/heads/master
Commit: 9629fa8fb9adadef90258b2edb96e46e8f9f3ceb
Parents: f77e032
Author: lidongsjtu <li...@apache.org>
Authored: Thu Aug 18 16:35:33 2016 +0800
Committer: lidongsjtu <li...@apache.org>
Committed: Thu Aug 18 16:35:33 2016 +0800

----------------------------------------------------------------------
 tool/pom.xml                                                    | 4 ----
 .../main/java/org/apache/kylin/tool/AbstractInfoExtractor.java  | 5 +++--
 2 files changed, 3 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/9629fa8f/tool/pom.xml
----------------------------------------------------------------------
diff --git a/tool/pom.xml b/tool/pom.xml
index 7561569..701fd13 100644
--- a/tool/pom.xml
+++ b/tool/pom.xml
@@ -49,10 +49,6 @@
             <artifactId>hbase-client</artifactId>
             <scope>provided</scope>
         </dependency>
-        <dependency>
-            <groupId>org.apache.kylin</groupId>
-            <artifactId>kylin-source-kafka</artifactId>
-        </dependency>
     </dependencies>
 
 

http://git-wip-us.apache.org/repos/asf/kylin/blob/9629fa8f/tool/src/main/java/org/apache/kylin/tool/AbstractInfoExtractor.java
----------------------------------------------------------------------
diff --git a/tool/src/main/java/org/apache/kylin/tool/AbstractInfoExtractor.java b/tool/src/main/java/org/apache/kylin/tool/AbstractInfoExtractor.java
index 233f994..66f8177 100644
--- a/tool/src/main/java/org/apache/kylin/tool/AbstractInfoExtractor.java
+++ b/tool/src/main/java/org/apache/kylin/tool/AbstractInfoExtractor.java
@@ -125,8 +125,9 @@ public abstract class AbstractInfoExtractor extends AbstractApplication {
     private void dumpBasicDiagInfo() throws IOException {
         try {
             for (String commitSHA1File : COMMIT_SHA1_FILES) {
-                if (new File(commitSHA1File).exists()) {
-                    FileUtils.copyFileToDirectory(new File(KylinConfig.getKylinHome(), commitSHA1File), exportDir);
+                File commitFile = new File(KylinConfig.getKylinHome(), commitSHA1File);
+                if (commitFile.exists()) {
+                    FileUtils.copyFileToDirectory(commitFile, exportDir);
                 }
             }
         } catch (IOException e) {