[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

[paul-rogers]

Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1032#discussion_r150685946
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java
 ---
    @@ -532,7 +572,10 @@ public boolean isMutable() {
         }
     
         public DrillFileSystem getFS() {
    -      return fs;
    +      if (this.fs == null) {
    +        this.fs = 
ImpersonationUtil.createFileSystem(schemaConfig.getUserName(), fsConf);
    +      }
    +      return this.fs;
    --- End diff --
    
    No need for `this.fs`, just `fs` will do.


---