Bikramjeet Vig has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11157


Change subject: IMPALA-7349: Add Admission control support for automatically 
setting mem_limit With this patch the mem_limit of a query is automatically set 
based on the mem_limit set in the query options and the mem_estimate calculated 
by the planner. This mem_limit wil
......................................................................

IMPALA-7349: Add Admission control support for automatically setting
mem_limit
With this patch the mem_limit of a query is automatically set based on
the mem_limit set in the query options and the mem_estimate calculated
by the planner. This mem_limit will also be used for admission
accounting and consequently for admission control. Moreover three new
pool configuration options have been added:
min-query-mem-limit-bytes, max-query-mem-limit-bytes => help clamp the
mem_limit.  If these are set to zero, impala will fall back to
traditional behavior (mem_limit not automatically set)
strict-min-max-query-mem-limit => if false, the mem_limit defined in
the query options is used directly and the max/min limits are not
enforced on it.

Testing:
Added e2e test cases.

Change-Id: Ifec00141651982f5975803c2165b7d7a10ebeaa6
---
M be/src/benchmarks/process-wide-locks-benchmark.cc
M be/src/runtime/coordinator-backend-state.cc
M be/src/runtime/coordinator.cc
M be/src/runtime/mem-tracker.cc
M be/src/runtime/mem-tracker.h
M be/src/runtime/query-exec-mgr.cc
M be/src/runtime/query-exec-mgr.h
M be/src/runtime/query-state.cc
M be/src/runtime/query-state.h
M be/src/runtime/runtime-state.cc
M be/src/runtime/test-env.cc
M be/src/scheduling/admission-controller.cc
M be/src/scheduling/admission-controller.h
M be/src/scheduling/query-schedule.cc
M be/src/scheduling/query-schedule.h
M be/src/scheduling/scheduler.cc
M be/src/service/client-request-state.cc
M common/thrift/ImpalaInternalService.thrift
M common/thrift/metrics.json
M fe/src/main/java/org/apache/impala/util/RequestPoolService.java
A fe/src/test/resources/fair-scheduler-mem-limit-test.xml
A fe/src/test/resources/llama-site-mem-limit-test.xml
A 
testdata/workloads/functional-query/queries/QueryTest/admission-max-min-mem-limits.test
M tests/custom_cluster/test_admission_controller.py
24 files changed, 858 insertions(+), 202 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/11157/6
--
To view, visit http://gerrit.cloudera.org:8080/11157
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifec00141651982f5975803c2165b7d7a10ebeaa6
Gerrit-Change-Number: 11157
Gerrit-PatchSet: 6
Gerrit-Owner: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>

Reply via email to