[4/8] kylin git commit: KYLIN-2217 fix bug

2016-11-24 Thread liyang 
KYLIN-2217 fix bug


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

Branch: refs/heads/KYLIN-2217-2
Commit: d9c8d9ec26b910f2a27f7a98dd44ef49e61fe416
Parents: 7f7417b
Author: Yang Li 
Authored: Fri Nov 25 07:16:45 2016 +0800
Committer: Li Yang 
Committed: Fri Nov 25 15:05:36 2016 +0800

--
 .../main/java/org/apache/kylin/engine/mr/DFSFileTable.java| 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/d9c8d9ec/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
--
diff --git 
a/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java 
b/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
index 8c1f6bd..6f337d0 100644
--- a/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
+++ b/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
@@ -61,12 +61,13 @@ public class DFSFileTable implements ReadableTable {
 
 @Override
 public TableSignature getSignature() throws IOException {
+Pair sizeAndLastModified;
 try {
-Pair sizeAndLastModified = 
getSizeAndLastModified(path);
-return new TableSignature(path, sizeAndLastModified.getFirst(), 
sizeAndLastModified.getSecond());
+sizeAndLastModified = getSizeAndLastModified(path);
 } catch (FileNotFoundException ex) {
-return null;
+sizeAndLastModified = Pair.newPair(-1L, 0L);
 }
+return new TableSignature(path, sizeAndLastModified.getFirst(), 
sizeAndLastModified.getSecond());
 }
 
 @Override

[3/3] kylin git commit: KYLIN-2217 fix bug

2016-11-24 Thread liyang 
KYLIN-2217 fix bug


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

Branch: refs/heads/KYLIN-2217-2
Commit: 7b50d3163456a748a98779b432161bde9da71b7d
Parents: 917a885
Author: Yang Li 
Authored: Fri Nov 25 07:16:45 2016 +0800
Committer: Li Yang 
Committed: Fri Nov 25 09:48:33 2016 +0800

--
 .../main/java/org/apache/kylin/engine/mr/DFSFileTable.java| 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/7b50d316/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
--
diff --git 
a/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java 
b/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
index 8c1f6bd..6f337d0 100644
--- a/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
+++ b/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
@@ -61,12 +61,13 @@ public class DFSFileTable implements ReadableTable {
 
 @Override
 public TableSignature getSignature() throws IOException {
+Pair sizeAndLastModified;
 try {
-Pair sizeAndLastModified = 
getSizeAndLastModified(path);
-return new TableSignature(path, sizeAndLastModified.getFirst(), 
sizeAndLastModified.getSecond());
+sizeAndLastModified = getSizeAndLastModified(path);
 } catch (FileNotFoundException ex) {
-return null;
+sizeAndLastModified = Pair.newPair(-1L, 0L);
 }
+return new TableSignature(path, sizeAndLastModified.getFirst(), 
sizeAndLastModified.getSecond());
 }
 
 @Override

kylin git commit: KYLIN-2217 fix bug

2016-11-24 Thread liyang 
Repository: kylin
Updated Branches:
  refs/heads/KYLIN-2217-2 e4a22531a -> d14de80ee


KYLIN-2217 fix bug


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

Branch: refs/heads/KYLIN-2217-2
Commit: d14de80ee17691f21ab9225bd1f2880ea1b17c04
Parents: e4a2253
Author: Yang Li 
Authored: Fri Nov 25 07:16:45 2016 +0800
Committer: Yang Li 
Committed: Fri Nov 25 07:16:45 2016 +0800

--
 .../main/java/org/apache/kylin/engine/mr/DFSFileTable.java| 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/d14de80e/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
--
diff --git 
a/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java 
b/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
index 8c1f6bd..6f337d0 100644
--- a/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
+++ b/engine-mr/src/main/java/org/apache/kylin/engine/mr/DFSFileTable.java
@@ -61,12 +61,13 @@ public class DFSFileTable implements ReadableTable {
 
 @Override
 public TableSignature getSignature() throws IOException {
+Pair sizeAndLastModified;
 try {
-Pair sizeAndLastModified = 
getSizeAndLastModified(path);
-return new TableSignature(path, sizeAndLastModified.getFirst(), 
sizeAndLastModified.getSecond());
+sizeAndLastModified = getSizeAndLastModified(path);
 } catch (FileNotFoundException ex) {
-return null;
+sizeAndLastModified = Pair.newPair(-1L, 0L);
 }
+return new TableSignature(path, sizeAndLastModified.getFirst(), 
sizeAndLastModified.getSecond());
 }
 
 @Override