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

    https://github.com/apache/drill/pull/877#discussion_r129103339
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/MetadataContext.java
 ---
    @@ -41,6 +41,10 @@
     
       private PruneStatus pruneStatus = PruneStatus.NOT_STARTED;
     
    +  // False values of these flags allow to avoid double reading of 
corrupted or unsupported metadata files
    +  public boolean isMetaCacheFileCorrect = true;
    +  public boolean isMetaDirsCacheFileCorrect = true;
    --- End diff --
    
    The state is created after failed opening of the metadata cache file. And 
will be stored per query.
    There is no case for now to clear that state manually. 
    Note: `MetadataContext.clear()` removes all metadata context info 
(`isMetadataFilesCorrupted` too).
    
    I left only one state, since if one file is unsupported or corrupted it is 
more likely that other files are the same. Even if not, it can be dangerous to 
read some files, but ignore the others.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to