Hello Philip Zeyliger, Dan Hecht, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9227 to look at the new patch set (#7). Change subject: IMPALA-6482: add EXEC_TIME_LIMIT_S option ...................................................................... IMPALA-6482: add EXEC_TIME_LIMIT_S option This is similar to the QUERY_TIMEOUT_S option and shares most of the implementation. The difference is that the timeout doesn't reset at any point. The time limit is measured from the start of query execution, after the query is admitted, so planning, scheduling and time spent in admission control is not counted towards the time limit. Also fix validation of the related QUERY_TIMEOUT_S option, which previously could ignore invalid input. Testing: Added tests for various permutations: * With and without query_timeout_s set * With and without result fetching keeping the query active Change-Id: Id81772ee223ffb64746e241027a5a734a811e1b8 --- M be/src/service/client-request-state.cc M be/src/service/client-request-state.h M be/src/service/impala-server.cc M be/src/service/impala-server.h M be/src/service/query-options-test.cc M be/src/service/query-options.cc M be/src/service/query-options.h M common/thrift/ImpalaInternalService.thrift M common/thrift/ImpalaService.thrift M tests/custom_cluster/test_query_expiration.py 10 files changed, 243 insertions(+), 91 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/27/9227/7 -- To view, visit http://gerrit.cloudera.org:8080/9227 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id81772ee223ffb64746e241027a5a734a811e1b8 Gerrit-Change-Number: 9227 Gerrit-PatchSet: 7 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>