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

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

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

    https://github.com/apache/drill/pull/738#discussion_r99713077
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java ---
    @@ -417,6 +417,9 @@ private void parseAndRunPhysicalPlan(final String json) 
throws ExecutionSetupExc
     
       private void runPhysicalPlan(final PhysicalPlan plan) throws 
ExecutionSetupException {
         validatePlan(plan);
    +    //Marking endTime of Planning
    +    queryManager.markPlanningEndTime();
    --- End diff --
    
    Nit. The next line is also part of planning, though should execute quite 
quickly.


> Display planning time for a query in its profile page
> -----------------------------------------------------
>
>                 Key: DRILL-5190
>                 URL: https://issues.apache.org/jira/browse/DRILL-5190
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Web Server
>    Affects Versions: 1.9.0
>            Reporter: Kunal Khatua
>            Assignee: Kunal Khatua
>
> Currently, the Web UI does not display the time spent in planning for a query 
> in its profile page. The estimate needs to be done by seeing how late did the 
> earliest major fragment start. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to