[GitHub] drill pull request #1024: DRILL-3640: Support JDBC Statement.setQueryTimeout...

[kkhatua]

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

    https://github.com/apache/drill/pull/1024#discussion_r149545534
  
    --- Diff: 
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java ---
    @@ -239,6 +261,11 @@ QueryDataBatch getNext() throws UserException, 
InterruptedException {
                 }
                 return qdb;
               }
    +
    +          // Check and throw SQLTimeoutException
    +          if ( parent.timeoutInMilliseconds > 0 && 
parent.elapsedTimer.elapsed(TimeUnit.SECONDS) >= parent.timeoutInMilliseconds ) 
{
    --- End diff --
    
    Darn! +1


---