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

    https://github.com/apache/drill/pull/1190#discussion_r177300882
  
    --- 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 --
    
    I refer to the entire comment. "If the request exceeds this timeout, then 
request is timed out" does not add any value, it is simply a definition for a 
timeout. The same for `CREATE_PREPARE_STATEMENT_TIMEOUT_MILLIS` and "Timeout 
for create prepare statement request".


---

Reply via email to