You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by cenyuhai <gi...@git.apache.org> on 2017/09/10 07:38:25 UTC

[GitHub] carbondata pull request #985: [CARBONDATA-1090] added integration test cases...

Github user cenyuhai commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/985#discussion_r137942968
  
    --- Diff: integration/hive/src/main/java/org/apache/carbondata/hive/CarbonDictionaryDecodeReadSupport.java ---
    @@ -231,11 +231,11 @@ private Writable createWritablePrimitive(Object obj, CarbonColumn carbonColumn)
           case LONG:
             return new LongWritable((long) obj);
           case SHORT:
    -        return new ShortWritable((Short) obj);
    +        return new ShortWritable((short) obj);
    --- End diff --
    
    why ？


---