KYLIN-1758 should check fact table

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

Branch: refs/heads/v1.5.2.1-release-rc1
Commit: 6688d6452103e4c7dfb71f36b096469c14c9904e
Parents: 0860934
Author: shaofengshi <shaofeng...@apache.org>
Authored: Fri Jun 3 18:18:04 2016 +0800
Committer: shaofengshi <shaofeng...@apache.org>
Committed: Fri Jun 3 18:31:28 2016 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/source/hive/HiveMRInput.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/6688d645/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
----------------------------------------------------------------------
diff --git 
a/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java 
b/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
index 9997b09..16642fa 100644
--- a/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
+++ b/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
@@ -156,7 +156,8 @@ public class HiveMRInput implements IMRInput {
             final Set<TableDesc> lookupViewsTables = Sets.newHashSet();
             for(DimensionDesc dimensionDesc : cubeDesc.getDimensions()) {
                 TableDesc tableDesc = dimensionDesc.getTableDesc();
-                if 
(TableDesc.TABLE_TYPE_VIRTUAL_VIEW.equalsIgnoreCase(tableDesc.getTableType())) {
+                if 
(TableDesc.TABLE_TYPE_VIRTUAL_VIEW.equalsIgnoreCase(tableDesc.getTableType())
+                        && 
!cubeDesc.getModel().getFactTableDesc().equals(tableDesc)) {
                     lookupViewsTables.add(tableDesc);
                 }
             }

Reply via email to