carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anubhav100 <...@git.apache.org>
Subject [GitHub] carbondata pull request #1356: [CARBONDATA-1477] corrected the bug for wrong...
Date Wed, 13 Sep 2017 15:43:06 GMT
Github user anubhav100 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1356#discussion_r138659199
  
    --- Diff: integration/hive/src/main/java/org/apache/carbondata/hive/CarbonDictionaryDecodeReadSupport.java
---
    @@ -228,9 +229,13 @@ 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);
           case DATE:
    -        return new DateWritable(new Date(((Integer) obj).longValue()));
    +        Calendar c = Calendar.getInstance();
    +        c.setTime(new Date(0));
    --- End diff --
    
    There are changes in master code for date so these changes are required for getting the
right values at hive side


---

Mime
View raw message