bug fix: remove checkDupByInfo in merge dict

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

Branch: refs/heads/2.x-staging
Commit: f7057e87c960e452cb18b3836712e58174f8501e
Parents: 280fce6
Author: honma <ho...@ebay.com>
Authored: Tue Jan 5 16:11:29 2016 +0800
Committer: honma <ho...@ebay.com>
Committed: Tue Jan 5 16:11:29 2016 +0800

----------------------------------------------------------------------
 .../java/org/apache/kylin/dict/DictionaryManager.java     | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/f7057e87/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryManager.java
----------------------------------------------------------------------
diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryManager.java 
b/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryManager.java
index f2bfd91..c718704 100644
--- a/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryManager.java
+++ b/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryManager.java
@@ -200,11 +200,11 @@ public class DictionaryManager {
         signature.setLastModifiedTime(System.currentTimeMillis());
         signature.setPath("merged_with_no_original_path");
 
-        String dupDict = checkDupByInfo(newDictInfo);
-        if (dupDict != null) {
-            logger.info("Identical dictionary input " + newDictInfo.getInput() 
+ ", reuse existing dictionary at " + dupDict);
-            return getDictionaryInfo(dupDict);
-        }
+//        String dupDict = checkDupByInfo(newDictInfo);
+//        if (dupDict != null) {
+//            logger.info("Identical dictionary input " + 
newDictInfo.getInput() + ", reuse existing dictionary at " + dupDict);
+//            return getDictionaryInfo(dupDict);
+//        }
 
         //check for cases where merging dicts are actually same
         boolean identicalSourceDicts = true;

Reply via email to