minor, fix findFristTable fact table filter in DataModelDesc
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/f1826704 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/f1826704 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/f1826704 Branch: refs/heads/KYLIN-2394 Commit: f18267043df55c2e8a6a378e126a4fafa302f3c8 Parents: 576d2dd Author: Billy Liu <billy...@apache.org> Authored: Tue Jan 17 13:38:39 2017 +0800 Committer: Billy Liu <billy...@apache.org> Committed: Tue Jan 17 13:38:39 2017 +0800 ---------------------------------------------------------------------- .../java/org/apache/kylin/metadata/model/DataModelDesc.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/f1826704/core-metadata/src/main/java/org/apache/kylin/metadata/model/DataModelDesc.java ---------------------------------------------------------------------- diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/model/DataModelDesc.java b/core-metadata/src/main/java/org/apache/kylin/metadata/model/DataModelDesc.java index 571b196..cc599a4 100644 --- a/core-metadata/src/main/java/org/apache/kylin/metadata/model/DataModelDesc.java +++ b/core-metadata/src/main/java/org/apache/kylin/metadata/model/DataModelDesc.java @@ -273,11 +273,16 @@ public class DataModelDesc extends RootPersistentEntity { if (rootFactTableRef.getTableIdentity().equals(tableIdentity)) return rootFactTableRef; + for (TableRef fact : factTableRefs) { + if (fact.getTableIdentity().equals(tableIdentity)) + return fact; + } + for (TableRef lookup : lookupTableRefs) { if (lookup.getTableIdentity().equals(tableIdentity)) return lookup; } - throw new IllegalArgumentException("Table not found by " + tableIdentity); + throw new IllegalArgumentException("Table not found by " + tableIdentity + " in model " + name); } public void init(KylinConfig config, Map<String, TableDesc> tables) {
