Repository: carbondata
Updated Branches:
  refs/heads/master c77221227 -> 8ea7272d9


[CARBONDATA-1407] Fix default end key bug for no-dictionary dimension

Now the default end key of no-dictionary dimension is 127 (0xEF), it should be 
0xFF.

This closes #1290


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

Branch: refs/heads/master
Commit: 8ea7272d9a80821f941f8fedfbade70ee8c678dd
Parents: c772212
Author: QiangCai <[email protected]>
Authored: Thu Aug 24 12:31:44 2017 +0800
Committer: Ravindra Pesala <[email protected]>
Committed: Wed Aug 30 14:49:22 2017 +0530

----------------------------------------------------------------------
 .../org/apache/carbondata/core/scan/filter/FilterUtil.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/carbondata/blob/8ea7272d/core/src/main/java/org/apache/carbondata/core/scan/filter/FilterUtil.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/carbondata/core/scan/filter/FilterUtil.java 
b/core/src/main/java/org/apache/carbondata/core/scan/filter/FilterUtil.java
index b952b97..39dee5a 100644
--- a/core/src/main/java/org/apache/carbondata/core/scan/filter/FilterUtil.java
+++ b/core/src/main/java/org/apache/carbondata/core/scan/filter/FilterUtil.java
@@ -1226,7 +1226,7 @@ public final class FilterUtil {
       startPoint++;
     }
     for (int i = 0; i < numberOfNoDictionaryDimension; i++) {
-      noDictionaryEndKeyBuffer.put((byte) 127);
+      noDictionaryEndKeyBuffer.put((byte) 0xFF);
     }
     return noDictionaryEndKeyBuffer.array();
   }
@@ -1481,7 +1481,7 @@ public final class FilterUtil {
         continue;
       }
       if (null == setOfStartKeyByteArray.get(dimension.getOrdinal())) {
-        setOfStartKeyByteArray.put(dimension.getOrdinal(), new byte[] { 127 });
+        setOfStartKeyByteArray.put(dimension.getOrdinal(), new byte[] { (byte) 
0xFF });
       }
 
     }
@@ -1513,7 +1513,7 @@ public final class FilterUtil {
       if (CarbonUtil.hasEncoding(dimension.getEncoder(), Encoding.DICTIONARY)) 
{
         continue;
       }
-      setOfEndKeyByteArray.put(dimension.getOrdinal(), new byte[] { 127 });
+      setOfEndKeyByteArray.put(dimension.getOrdinal(), new byte[] { (byte) 
0xFF });
     }
   }
 

Reply via email to