Github user laurentgo commented on a diff in the pull request:
https://github.com/apache/drill/pull/1024#discussion_r149477955
--- Diff:
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java ---
@@ -376,6 +415,19 @@ synchronized void cleanup() {
currentBatchHolder.clear();
}
+ //Set the cursor's timeout in seconds
--- End diff --
you just need to get the value when the query is executed (in DrillCursor)
once to make sure the timeout doesn't change (that and StopWatch being managed
by DrillCursor too.
Also, it is subject to interpretation but it seems the intent of the API is
to time bound how much time it takes the query to complete. I'm not sure it is
necessary to make the extra work of having a slow client reading the result set
data although all data has already been read by the driver from the server (and
from the server point of view, the query is completed).
---
