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

    https://github.com/apache/carbondata/pull/2374#discussion_r196625604
  
    --- Diff: 
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonFileInputFormat.java
 ---
    @@ -174,9 +174,15 @@ public CarbonTable 
getOrCreateCarbonTable(Configuration configuration) throws IO
         List<InputSplit> result = new LinkedList<InputSplit>();
     
         // for each segment fetch blocks matching filter in Driver BTree
    -    List<CarbonInputSplit> dataBlocksOfSegment =
    -        getDataBlocksOfSegment(job, carbonTable, filterResolver, 
matchedPartitions,
    -            validSegments, partitionInfo, oldPartitionIdList);
    +    List<CarbonInputSplit> dataBlocksOfSegment;
    +    if (carbonTable.getTableInfo().getFormat().equals("")
    --- End diff --
    
    The default value of format is 'carbondata', so there is no need to handle 
empty. Will remove it


---

Reply via email to