You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by vrozov <gi...@git.apache.org> on 2018/03/26 14:17:16 UTC

[GitHub] drill pull request #1190: DRILL-5937: ExecConstants: changed comment, timeou...

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

    https://github.com/apache/drill/pull/1190#discussion_r177103076
  
    --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
    @@ -558,7 +558,7 @@ private ExecConstants() {
     
       /**
        * Timeout for create prepare statement request. If the request exceeds this timeout, then request is timed out.
    -   * Default value is 10mins.
    +   * Default value is 30 seconds.
    --- End diff --
    
    Is it 30 seconds or 10 seconds? In any case, I don't think that the entire comment provides any additional info that cannot be inferred from the code itself.


---