[GitHub] carbondata pull request #2214: [CARBONDATA-2387]Primitive AVRO Datatype Chec...

[jackylk]

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

    https://github.com/apache/carbondata/pull/2214#discussion_r183374676
  
    --- Diff: 
store/sdk/src/main/java/org/apache/carbondata/sdk/file/AvroCarbonWriter.java ---
    @@ -85,11 +85,15 @@ private String avroFieldToString(Schema.Field 
fieldType, Object fieldValue) {
           case LONG:
           case DOUBLE:
           case STRING:
    +      case FLOAT:
             out.append(fieldValue.toString());
             break;
    +      case NULL:
    +        out.append("");
    --- End diff --
    
    In testcase it is saying NULL is not supported, but here it is ""?


---