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

    https://github.com/apache/drill/pull/755#discussion_r171945885
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/store/LocalPersistentStore.java
 ---
    @@ -112,23 +127,65 @@ public static DrillFileSystem 
getFileSystem(DrillConfig config, Path root) throw
     
       @Override
       public Iterator<Map.Entry<String, V>> getRange(int skip, int take) {
    +    //Marking currently seen modification time
    +    long currBasePathModified = 0L;
    +    try {
    +      currBasePathModified = 
fs.getFileStatus(basePath).getModificationTime();
    +    } catch (IOException ioexcp) {
    +      ioexcp.printStackTrace();
    +    }
    +
    +    //Acquiring lock to avoid reloading for request coming in before 
completion of profile read
    --- End diff --
    
    I'll provide the explanation below.


---

Reply via email to