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`
---