Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12407 )
Change subject: IMPALA-5043: diagnostics for topic staleness in AC ...................................................................... IMPALA-5043: diagnostics for topic staleness in AC The default threshold for the admission control topic to be considered stale is 5s. Adds diagnostics for stale topic updates: * A banner on the /admission web UI if the topic is considered stale. * Time since last update on the /admission web UI * Append a warning to rejection/queuing messages where topic staleness may have affected the decision. * Append a warning to profiles of admitted queries where the topic was stale at the time the query was admitted. * Include the topic staleness in all profiles of admitted queries Testing: Add a custom cluster test that kills the statestore, validates that admission control behaves as expected and that stateless warnings show up in the appropriate places. Change-Id: Ib9e26adb6419589ccf7625e423356df45bee4ac9 Reviewed-on: http://gerrit.cloudera.org:8080/12407 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/scheduling/admission-controller.cc M be/src/scheduling/admission-controller.h M be/src/service/impala-http-handler.cc M tests/common/impala_test_suite.py M tests/custom_cluster/test_admission_controller.py M www/admission_controller.tmpl 6 files changed, 208 insertions(+), 34 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/12407 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib9e26adb6419589ccf7625e423356df45bee4ac9 Gerrit-Change-Number: 12407 Gerrit-PatchSet: 6 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
