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

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

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

    https://github.com/apache/drill/pull/719#discussion_r95692761
  
    --- Diff: exec/java-exec/src/main/resources/rest/profile/profile.ftl ---
    @@ -106,6 +106,7 @@
       <p>STATE: ${model.getProfile().getState().name()}</p>
       <p>FOREMAN: ${model.getProfile().getForeman().getAddress()}</p>
       <p>TOTAL FRAGMENTS: ${model.getProfile().getTotalFragments()}</p>
    +  <p>DURATION: ${model.getProfileDuration()}</p>
    --- End diff --
    
    The end time for a query is not set until the very end of the query, so the 
default value in 0L. For the `/profiles ` template, the elapsed time is 
computed using the current time in millis for running queries. Hence it shows 
correctly here. As per your suggestion, fixed it now by changing the signature 
to take the start and end times as arguments.  


> Display elapsed time for queries in the UI
> ------------------------------------------
>
>                 Key: DRILL-5172
>                 URL: https://issues.apache.org/jira/browse/DRILL-5172
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Web Server
>    Affects Versions: 1.9.0
>            Reporter: Kunal Khatua
>            Assignee: Kunal Khatua
>            Priority: Minor
>             Fix For: 1.10.0
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Currently, the Web UI does not display the runtime for a query either in the 
> list of queries or the query profile page itself.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to