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

    https://github.com/apache/carbondata/pull/2262#discussion_r187314011
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/readcommitter/TableStatusReadCommittedScope.java
 ---
    @@ -77,6 +78,12 @@ public 
TableStatusReadCommittedScope(AbsoluteTableIdentifier identifier,
         return indexFiles;
       }
     
    +  public Map<String, SegmentLatestTimestampUpdater> 
getCommitedSegmentTimestampUpdaterMap(
    --- End diff --
    
    You should not return empty map here. You should pass updateManager through 
constructor and get the update timestamp from it and return new 
SegmentLatestTimestampUpdater


---

Reply via email to