Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/993#discussion_r145565896
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRestServer.java
---
@@ -204,9 +217,15 @@ public WebUserConnection provide() {
config.getLong(ExecConstants.HTTP_SESSION_MEMORY_RESERVATION),
config.getLong(ExecConstants.HTTP_SESSION_MEMORY_MAXIMUM));
+ // Create a dummy close future which is needed by Foreman only.
Foreman uses this future to add a close
+ // listener to known about channel close event from underlying
layer. We use this future to notify Foreman
+ // listeners when the Web connection between Web Client and
WebServer is closed. This will help Foreman to cancel
+ // all the running queries for this Web Client.
--- End diff --
The query is killed when the HttpSession is invalidated either by logout or
by timeout in case of authenticated user session. But for unauthenticated or
anonymous user we don't kill the query after HttpSession timeout since we don't
maintain any session related information. The query will keep running unless it
is explicitly cancelled from /profile page.
---
