Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16039 )
Change subject: IMPALA-9077: Remove scalable admission control configs ...................................................................... IMPALA-9077: Remove scalable admission control configs Removed the 3 scalable configs added in IMPALA-8536: - Max Memory Multiple - Max Running Queries Multiple - Max Queued Queries Multiple This patch removes the functionality related to those configs but retains the additional test coverage and cleanup added in IMPALA-8536. This removal is to make it easier to enhance Admission Control using Executor Groups which has turned out to be a useful building block. Testing: Ran core tests. Change-Id: Ib9bd63f03758a6c4eebb99c64ee67e60cb56b5ac Reviewed-on: http://gerrit.cloudera.org:8080/16039 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/scheduling/admission-controller-test.cc M be/src/scheduling/admission-controller.cc M be/src/scheduling/admission-controller.h M common/thrift/ImpalaInternalService.thrift M common/thrift/metrics.json M fe/src/main/java/org/apache/impala/util/RequestPoolService.java M fe/src/test/java/org/apache/impala/util/TestRequestPoolService.java M fe/src/test/resources/fair-scheduler-test.xml M fe/src/test/resources/fair-scheduler-test2.xml M fe/src/test/resources/llama-site-test.xml M fe/src/test/resources/llama-site-test2.xml M testdata/workloads/functional-query/queries/QueryTest/admission-max-min-mem-limits.test M testdata/workloads/functional-query/queries/QueryTest/admission-reject-mem-estimate.test M testdata/workloads/functional-query/queries/QueryTest/admission-reject-min-reservation.test M tests/custom_cluster/test_admission_controller.py M tests/webserver/test_web_pages.py M www/admission_controller.tmpl 17 files changed, 158 insertions(+), 795 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16039 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib9bd63f03758a6c4eebb99c64ee67e60cb56b5ac Gerrit-Change-Number: 16039 Gerrit-PatchSet: 9 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
