[GitHub] carbondata pull request #1793: [CARBONDATA-2021]fix clean up issue when upda...

[jackylk]

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

    https://github.com/apache/carbondata/pull/1793#discussion_r161987134
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentUpdateStatusManager.java
 ---
    @@ -956,9 +964,16 @@ public UpdateVO getInvalidTimestampRange(String 
segmentId) {
             long timestamp = CarbonUpdateUtil.getTimeStampAsLong(
                 
CarbonTablePath.DataFileUtil.getTimeStampFromDeleteDeltaFile(fileName));
     
    -        if (block.getBlockName().equalsIgnoreCase(blkName) && (
    -            Long.compare(timestamp, deltaStartTimestamp) < 0)) {
    -          files.add(eachFile);
    +        if (block.getBlockName().equalsIgnoreCase(blkName)) {
    +
    +          if (getOnlyAbortedFiles) {
    +            if (Long.compare(timestamp, deltaEndTimestamp) > 0) {
    --- End diff --
    
    Can't you use timestamp > deltaEndTimestamp?


---