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

    https://github.com/apache/carbondata/pull/1994#discussion_r170244941
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/DeleteLoadFolders.java ---
    @@ -188,8 +188,11 @@ public static boolean deleteLoadFoldersFromFileSystem(
                   LOGGER.info("Info: Deleted the load " + 
oneLoad.getLoadName());
                 }
               } finally {
    -            segmentLock.unlock();
    -            LOGGER.info("Info: Segment lock on segment:" + 
oneLoad.getLoadName() + " is released");
    +            if (segmentLock.isLocked()) {
    --- End diff --
    
    FYI:
    Only unlock if we lock it successfully, otherwise this will cause failure 
in windows.


---

Reply via email to