[ 
https://issues.apache.org/jira/browse/DRILL-5089?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16257240#comment-16257240
 ] 

ASF GitHub Bot commented on DRILL-5089:
---------------------------------------

Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1032#discussion_r151732963
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java
 ---
    @@ -373,12 +402,12 @@ public String toString() {
       public class WorkspaceSchema extends AbstractSchema implements 
ExpandingConcurrentMap.MapValueFactory<TableInstance, DrillTable> {
         private final ExpandingConcurrentMap<TableInstance, DrillTable> tables 
= new ExpandingConcurrentMap<>(this);
         private final SchemaConfig schemaConfig;
    -    private final DrillFileSystem fs;
    +    private DrillFileSystem fs;
     
    -    public WorkspaceSchema(List<String> parentSchemaPath, String wsName, 
SchemaConfig schemaConfig) throws IOException {
    +    public WorkspaceSchema(List<String> parentSchemaPath, String wsName, 
SchemaConfig schemaConfig, DrillFileSystem fs) throws IOException {
           super(parentSchemaPath, wsName);
           this.schemaConfig = schemaConfig;
    -      this.fs = 
ImpersonationUtil.createFileSystem(schemaConfig.getUserName(), fsConf);
    +      this.fs = fs;
    --- End diff --
    
    Why don't we anymore need to create fs using `ImpersonationUtil` but needed 
before?


> Skip initializing all enabled storage plugins for every query
> -------------------------------------------------------------
>
>                 Key: DRILL-5089
>                 URL: https://issues.apache.org/jira/browse/DRILL-5089
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Query Planning & Optimization
>    Affects Versions: 1.9.0
>            Reporter: Abhishek Girish
>            Assignee: Chunhui Shi
>            Priority: Critical
>
> In a query's lifecycle, at attempt is made to initialize each enabled storage 
> plugin, while building the schema tree. This is done regardless of the actual 
> plugins involved within a query. 
> Sometimes, when one or more of the enabled storage plugins have issues - 
> either due to misconfiguration or the underlying datasource being slow or 
> being down, the overall query time taken increases drastically. Most likely 
> due the attempt being made to register schemas from a faulty plugin.
> For example, when a jdbc plugin is configured with SQL Server, and at one 
> point the underlying SQL Server db goes down, any Drill query starting to 
> execute at that point and beyond begin to slow down drastically. 
> We must skip registering unrelated schemas (& workspaces) for a query. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to