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

    https://github.com/apache/carbondata/pull/1608#discussion_r154835950
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java
 ---
    @@ -493,6 +493,19 @@ public static void writeLoadDetailsIntoFile(String 
dataLoadLocation,
                 invalidLoadIds.add(loadId);
                 return invalidLoadIds;
               }
    +          // if the segment status is in progress then no need to delete 
that.
    +          if (SegmentStatus.INSERT_IN_PROGRESS == 
loadMetadata.getSegmentStatus()) {
    +            LOG.error("Cannot delete the Segment which is In Progress. 
Segment is " + loadId);
    --- End diff --
    
    Please rephrase the sentence 
    `"Cannot delete the segemnt "+ loadId +" which is load in progress" `


---

Reply via email to