Pooja Nilangekar has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12583 )
Change subject: IMPALA-5397: Set query's end_time_us_ when the operation completes ...................................................................... IMPALA-5397: Set query's end_time_us_ when the operation completes Previously, a query's end time was set only when UnregisterQuery() was called. This was misleading because the query could have completed earlier. After this change, if the query has a coordinator, the end time is set when the query releases its admission control resources. For queries without coordinators, (e.g. DDL queries), the end time would still be set when UnregisterQuery() is called. Testing: Added a test to ensure that the end time is set before the query is closed. Tested queries with various settings of --idle-query-timeout. Ran exhaustive end-to-end tests and the stress test without any failures. Change-Id: Iaa9a1b443df3dbd95f9f297c2f923ad795b14745 --- M be/src/runtime/coordinator.cc M be/src/runtime/coordinator.h M be/src/service/client-request-state.cc M be/src/service/client-request-state.h M tests/common/impala_test_suite.py M tests/query_test/test_observability.py 6 files changed, 56 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/83/12583/3 -- To view, visit http://gerrit.cloudera.org:8080/12583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iaa9a1b443df3dbd95f9f297c2f923ad795b14745 Gerrit-Change-Number: 12583 Gerrit-PatchSet: 3 Gerrit-Owner: Pooja Nilangekar <pooja.nilange...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Pooja Nilangekar <pooja.nilange...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>