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

Posted to issues@carbondata.apache.org by gvramana <gi...@git.apache.org> on 2017/09/20 11:07:41 UTC

[GitHub] carbondata pull request #1101: [CARBONDATA-1143] fix for null struct type

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

    https://github.com/apache/carbondata/pull/1101#discussion_r139939203
  
    --- Diff: processing/src/main/java/org/apache/carbondata/processing/datatypes/StructDataType.java ---
    @@ -145,7 +146,7 @@ public void setSurrogateIndex(int surrIndex) {
           throws IOException, DictionaryGenerationException {
         dataOutputStream.writeInt(children.size());
         if (input == null) {
    -      dataOutputStream.writeInt(children.size());
    +      // dataOutputStream.writeInt(children.size());
    --- End diff --
    
    remove the comment


---