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

    https://github.com/apache/carbondata/pull/2900#discussion_r231012866
  
    --- Diff: 
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java
 ---
    @@ -575,6 +576,8 @@ private BitSet setMatchedPartitions(String 
partitionIds, Expression filter,
        */
       public BlockMappingVO getBlockRowCount(Job job, CarbonTable table,
           List<PartitionSpec> partitions) throws IOException {
    +    // no useful information for count star query without filter, so 
disable explain collector
    +    ExplainCollector.remove();
    --- End diff --
    
    oh... I understand.
    The current implementation of pruning collector may has bugs. Based on the 
current implementation, your modification is OK...


---

Reply via email to