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

    https://github.com/apache/carbondata/pull/2262#discussion_r187309701
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java 
---
    @@ -424,7 +425,7 @@ public TableSegmentRefresher 
getTableSegmentRefresher(CarbonTable table) {
     
         // This map stores the latest segment refresh time.So in case of 
update/delete we check the
         // time against this map.
    -    private Map<String, Long> segmentRefreshTime = new HashMap<>();
    +    private Map<String, SegmentLatestTimestampUpdater> segmentRefreshTime 
= new HashMap<>();
    --- End diff --
    
    Better rename as `SegmentRefreshInfo`


---

Reply via email to