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

    https://github.com/apache/carbondata/pull/2978#discussion_r240526032
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/scan/result/vector/impl/CarbonColumnVectorImpl.java
 ---
    @@ -367,7 +378,14 @@ public void setBlockDataType(DataType blockDataType) {
       }
     
       @Override public void setLazyPage(LazyPageLoader lazyPage) {
    -    lazyPage.loadPage();
    +    this.lazyPage = lazyPage;
    +  }
    +
    +  public void loadPage() {
    +    if (lazyPage != null) {
    +      lazyPage.loadPage();
    +    }
    +    loaded = true;
    --- End diff --
    
    if lazyPath is null, still set to true?


---

Reply via email to