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

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

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

    https://github.com/apache/drill/pull/1062#discussion_r155875821
  
    --- Diff: exec/java-exec/src/main/resources/rest/index.ftl ---
    @@ -185,11 +185,10 @@
           </div>
       </div>
        <script charset="utf-8">
    -      var refreshTime = 2000;
    +      var refreshTime = 10000;
           var refresh = getRefreshTime();
    --- End diff --
    
    This is confusing isn't it? The function does not return a value and yet 
the return value (<undefined>) is stored in a variable that is not used. 
    Luckily it's not used anywhere. 



> web ui index page is refreshed at high pace 
> --------------------------------------------
>
>                 Key: DRILL-6007
>                 URL: https://issues.apache.org/jira/browse/DRILL-6007
>             Project: Apache Drill
>          Issue Type: Bug
>    Affects Versions: 1.12.0
>            Reporter: Arina Ielchiieva
>            Assignee: Arina Ielchiieva
>            Priority: Blocker
>             Fix For: 1.12.0
>
>         Attachments: insufficient_resources.JPG
>
>
> After DRILL-4286 Web UI index page is constantly refreshed to get up-to-date 
> drillbits status. When graceperiod is set to 0, it is refreshed at high pace 
> because refresh time is counted as [gracepeiod / 
> 3|https://github.com/apache/drill/blob/master/exec/java-exec/src/main/resources/rest/index.ftl#L215].
>  Sometimes I saw the insufficient resources error 
> (insufficient_resources.JPG).
> Also 
> [graceperiod|https://github.com/apache/drill/blob/master/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRoot.java#L106]
>  should be in camel case.
> Fix:
> use default (10 seconds) if grace period is set to 0.



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

Reply via email to