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

    https://github.com/apache/carbondata/pull/2703#discussion_r217060418
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/indexstore/BlockletDetailInfo.java
 ---
    @@ -51,7 +51,8 @@
     
       private short versionNumber;
     
    -  private short blockletId;
    +  // default blockletId should be -1,which means consider all the 
blocklets in block
    --- End diff --
    
    @xuchuanyin ,default  cache level is Block and for block we always set 
blockletid as -1. so it is better to handle during initialization rather than 
handle in each reader (if any new reader implemented in future then it will be 
taken care automatically otherwise each reader should take care same ). 


---

Reply via email to