You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/06/25 23:57:06 UTC

kylin git commit: KYLIN-1821 fix CI, NPE in MeasureDesc.hashCode()

Repository: kylin
Updated Branches:
  refs/heads/master 1a124e68f -> e9a36c6c5


KYLIN-1821 fix CI, NPE in MeasureDesc.hashCode()


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

Branch: refs/heads/master
Commit: e9a36c6c5d47b67ab1a8b0bd3361a77001e06080
Parents: 1a124e6
Author: Yang Li <li...@apache.org>
Authored: Sun Jun 26 07:56:44 2016 +0800
Committer: Yang Li <li...@apache.org>
Committed: Sun Jun 26 07:56:51 2016 +0800

----------------------------------------------------------------------
 .../main/java/org/apache/kylin/metadata/model/MeasureDesc.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/e9a36c6c/core-metadata/src/main/java/org/apache/kylin/metadata/model/MeasureDesc.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/model/MeasureDesc.java b/core-metadata/src/main/java/org/apache/kylin/metadata/model/MeasureDesc.java
index 7abfad1..dcf6350 100644
--- a/core-metadata/src/main/java/org/apache/kylin/metadata/model/MeasureDesc.java
+++ b/core-metadata/src/main/java/org/apache/kylin/metadata/model/MeasureDesc.java
@@ -63,7 +63,7 @@ public class MeasureDesc {
 
     @Override
     public int hashCode() {
-        return Objects.hash(function, dependentMeasureRef.toLowerCase());
+        return Objects.hash(function, dependentMeasureRef);
     }
 
     @Override
@@ -84,7 +84,7 @@ public class MeasureDesc {
         if (dependentMeasureRef == null && that.getDependentMeasureRef() == null)
             return true;
 
-        return dependentMeasureRef.equalsIgnoreCase(that.getDependentMeasureRef());
+        return dependentMeasureRef.equals(that.getDependentMeasureRef());
     }
 
     @Override