Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-2198: Differentiate queries in exceptional states in web UI ......................................................................
IMPALA-2198: Differentiate queries in exceptional states in web UI In order to make the query life-cycle clearer to users, added a new section to the /queries webui page for queries that are 'waiting', not actively running either due to an error or to returning all of their results, but that have not been closed so they are still using resources. This section is marked 'waiting to be closed' to indicate that they still need to be closed even though they are not actively running. These queries previously would have appeared in the 'in flight' list. There is a tooltip with a full explanation. The 'in_flight_queries' json endpoint was left as is, so that CM will continue to work as expected, and filtering queries for the different lists is done in the html template. This was tested manually. Change-Id: I47d0b642ecb573fefbbf337b8c8f2c479b0d49b2 Reviewed-on: http://gerrit.cloudera.org:8080/2625 Reviewed-by: Matthew Jacobs <[email protected]> Tested-by: Internal Jenkins --- M be/src/service/impala-server-callbacks.cc M be/src/service/impala-server.cc M be/src/service/impala-server.h M tests/common/impala_service.py M tests/custom_cluster/test_query_expiration.py A tests/hs2/test_json_endpoints.py M www/queries.tmpl 7 files changed, 188 insertions(+), 1 deletion(-) Approvals: Matthew Jacobs: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2625 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I47d0b642ecb573fefbbf337b8c8f2c479b0d49b2 Gerrit-PatchSet: 13 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
