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

    https://github.com/apache/drill/pull/1146#discussion_r171912800
  
    --- Diff: 
contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveDrillNativeScanBatchCreator.java
 ---
    @@ -174,7 +174,7 @@ public ScanBatch getBatch(ExecutorFragmentContext 
context, HiveDrillNativeParque
         // If there are no readers created (which is possible when the table 
is empty or no row groups are matched),
         // create an empty RecordReader to output the schema
         if (readers.size() == 0) {
    -      readers.add(new HiveDefaultReader(table, null, null, columns, 
context, conf,
    +      readers.add(new HiveDefaultReader(table, null, null, newColumns, 
context, conf,
    --- End diff --
    
    Could we rename newColumns -> nonPartitionedColumns or tableColumns?


---

Reply via email to