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

    https://github.com/apache/carbondata/pull/2262#discussion_r187376048
  
    --- 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, SegmentRefreshInfo> 
getCommitedSegmentTimestampUpdaterMap(
    +      Segment segment) throws IOException {
    +    Map<String, SegmentRefreshInfo> indexFiles = new HashMap<>();
    +    return indexFiles;
    --- End diff --
    
    It should return only SegmentRefreshInfo object not Map.
    And it should be fillied with updateVo timestamp.


---

Reply via email to