You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by ma...@apache.org on 2016/11/19 14:21:18 UTC
[08/12] kylin git commit: KYLIN-2199 bug fix
KYLIN-2199 bug fix
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/d87b7a51
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/d87b7a51
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/d87b7a51
Branch: refs/heads/yang21-cdh5.7
Commit: d87b7a51282ffb2e678af65c20b5a395a2347d49
Parents: 9bd6a2a
Author: Hongbin Ma <ma...@apache.org>
Authored: Thu Nov 17 16:46:59 2016 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Thu Nov 17 16:48:53 2016 +0800
----------------------------------------------------------------------
.../org/apache/kylin/dimension/DimensionEncodingFactory.java | 6 ++++++
1 file changed, 6 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/d87b7a51/core-metadata/src/main/java/org/apache/kylin/dimension/DimensionEncodingFactory.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/dimension/DimensionEncodingFactory.java b/core-metadata/src/main/java/org/apache/kylin/dimension/DimensionEncodingFactory.java
index 242e003..b617a19 100644
--- a/core-metadata/src/main/java/org/apache/kylin/dimension/DimensionEncodingFactory.java
+++ b/core-metadata/src/main/java/org/apache/kylin/dimension/DimensionEncodingFactory.java
@@ -73,6 +73,12 @@ public abstract class DimensionEncodingFactory {
Map<String, Integer> result = Maps.newHashMap();
for (Pair<String, Integer> p : factoryMap.keySet()) {
+ if (result.containsKey(p.getFirst())) {
+ if (result.get(p.getFirst()) > p.getSecond()) {
+ continue;//skip small versions
+ }
+ }
+
result.put(p.getFirst(), p.getSecond());
}
result.put(DictionaryDimEnc.ENCODING_NAME, 1);