carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ravipesala <...@git.apache.org>
Subject [GitHub] carbondata pull request #1265: [CARBONDATA-1128] Add encoding for non-dictio...
Date Thu, 07 Sep 2017 05:43:31 GMT
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1265#discussion_r137450241
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/scan/filter/partition/EqualToFilterImpl.java
---
    @@ -50,7 +50,7 @@ public EqualToFilterImpl(EqualToExpression equalTo, PartitionInfo partitionInfo)
               literal.getLiteralExpValue().toString(),
               partitionInfo.getColumnSchemaList().get(0).getDataType());
           if (PartitionType.RANGE == partitionInfo.getPartitionType() && value instanceof
String) {
    -        value = ByteUtil.toBytes((String)value);
    +        value = ByteUtil.toBytesForPlainValue((String)value);
    --- End diff --
    
    It seems old overloaded method looks good than this name. It is utility method so better
have a overloaded method


---

Mime
View raw message