You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by "Jacques Nadeau (JIRA)" <ji...@apache.org> on 2014/04/20 03:37:14 UTC

[jira] [Commented] (DRILL-384) Move some Fragment initialization work to run() method

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

Jacques Nadeau commented on DRILL-384:
--------------------------------------

please rebase on latest master

> Move some Fragment initialization work to run() method
> ------------------------------------------------------
>
>                 Key: DRILL-384
>                 URL: https://issues.apache.org/jira/browse/DRILL-384
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Steven Phillips
>            Assignee: Steven Phillips
>         Attachments: DRILL-384.patch
>
>
> Currently, when initializing fragments for execution, ImplCreator.getExec() is run on the fragment plan, and the result RootExec object is passed to the constructor of the FragmentExecutor. This work all happens on an rpc thread, and can include setting up readers, which involves reading metadata from disk. It would be best to move this work to the run() method of the FragmentExecutor, since that method is called from a work thread.
> This will help speed up the initialization of fragments, especially when there are a large number of them.



--
This message was sent by Atlassian JIRA
(v6.2#6252)