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

    https://github.com/apache/carbondata/pull/2533#discussion_r204331739
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockDataMap.java
 ---
    @@ -97,10 +98,15 @@
        * partition table and non transactional table
        */
       protected boolean isFilePathStored;
    +  /**
    +   * datamap model
    +   */
    +  protected BlockletDataMapModel blockletDataMapModel;
    --- End diff --
    
    Don't take complete `blockletDataMapModel` to class level, it wastes memory 
as we keep them in LRU . Just keep boolean isStandardCarbonTable in class level


---

Reply via email to