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

    https://github.com/apache/drill/pull/1049#discussion_r153735994
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ColumnReaderFactory.java
 ---
    @@ -138,6 +138,8 @@
                     return new 
ParquetFixedWidthDictionaryReaders.DictionaryBigIntReader(recordReader, 
allocateSize, descriptor, columnChunkMetaData, fixedLength, (BigIntVector) v, 
schemaElement);
    --- End diff --
    
    This comment had to be placed in line 
[117](https://github.com/apache/drill/pull/1049/files?diff=unified#diff-4a7ec07122bfb16e4ff696af256f56dcR117),
 but I could not add it there. 
    Should we also handle the case when `columnChunkMetaData.getType()` type is 
`INT32` and `convertedType` is `INT_32`? 


---

Reply via email to