Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1138#discussion_r171607241
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/avro/AvroRecordReader.java
 ---
    @@ -295,7 +301,8 @@ private void processPrimitive(final Object value, final 
Schema.Type type, final
             writer.binary(fieldName).writeVarBinary(0, length, buffer);
             break;
           case NULL:
    -        // Nothing to do for null type
    +        // The default Drill behaviour is to create int column
    +        writer.integer(fieldName);
    --- End diff --
    
    This maps a NULL type to integer. Probably OK if we do this consistently.


---

Reply via email to