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 2015/12/27 08:13:33 UTC
kylin git commit: KYLIN-976 Minor change for DataType validation
Repository: kylin
Updated Branches:
refs/heads/2.x-staging e58326999 -> 11d5dc90c
KYLIN-976 Minor change for DataType validation
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/11d5dc90
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/11d5dc90
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/11d5dc90
Branch: refs/heads/2.x-staging
Commit: 11d5dc90c21364eb6cdd8b1da353553056d4fa02
Parents: e583269
Author: lidongsjtu <do...@ebay.com>
Authored: Sun Dec 27 15:12:36 2015 +0800
Committer: lidongsjtu <do...@ebay.com>
Committed: Sun Dec 27 15:12:36 2015 +0800
----------------------------------------------------------------------
.../src/main/java/org/apache/kylin/measure/MeasureTypeFactory.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/11d5dc90/core-metadata/src/main/java/org/apache/kylin/measure/MeasureTypeFactory.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/measure/MeasureTypeFactory.java b/core-metadata/src/main/java/org/apache/kylin/measure/MeasureTypeFactory.java
index 06645a9..5c2e6ed 100644
--- a/core-metadata/src/main/java/org/apache/kylin/measure/MeasureTypeFactory.java
+++ b/core-metadata/src/main/java/org/apache/kylin/measure/MeasureTypeFactory.java
@@ -106,7 +106,7 @@ abstract public class MeasureTypeFactory<T> {
String funcName = factory.getAggrFunctionName();
if (funcName.equals(funcName.toUpperCase()) == false)
throw new IllegalArgumentException("Aggregation function name '" + funcName + "' must be in upper case");
- String dataTypeName = factory.getAggrDataTypeName().toLowerCase();
+ String dataTypeName = factory.getAggrDataTypeName();
if (dataTypeName.equals(dataTypeName.toLowerCase()) == false)
throw new IllegalArgumentException("Aggregation data type name '" + dataTypeName + "' must be in lower case");
Class<? extends DataTypeSerializer<?>> serializer = factory.getAggrDataTypeSerializer();