Anubhav Jindal has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/24965


Change subject: IMPALA-13965: Enforce quotas when no executors are available
......................................................................

IMPALA-13965: Enforce quotas when no executors are available

Admission could return early when no executor group was available, skipping
quota rejection on submission and allowing queued requests to overflow user
limits until executors came online. This change enforces quota in the
empty-group path and tracks queued user-load ownership so reject/timeout/cancel
paths decrement per-user load exactly once.

Adds regression coverage in admission-controller unit tests and
custom-cluster executor-group tests for both no-executor quota enforcement and
queued-timeout load cleanup.

Testing:
- ssh [email protected] "cd 
/home/anubhav/impala-worktrees/IMPALA-12232-stack && ninja -C $PWD -j 
${IMPALA_BUILD_THREADS} impalad statestored catalogd admission-controller-test"
- ssh [email protected] "cd 
/home/anubhav/impala-worktrees/IMPALA-12232-stack && 
be/build/debug/scheduling/admission-controller-test 
--gtest_filter=AdmissionControllerTest.UserAndGroupQuotas:AdmissionControllerTest.PoolStats"
- ssh [email protected] "cd 
/home/anubhav/impala-worktrees/IMPALA-12232-stack && bin/impala-py.test 
tests/custom_cluster/test_executor_groups.py -k 
'user_quota_applies_without_executors or timed_out_queue_releases_user_quota'"

Change-Id: I84024a8d11911b639679029e34445ca52902ab93
Assisted-by: GPT-5.3 (Cursor)
---
M be/src/scheduling/admission-controller-test.cc
M be/src/scheduling/admission-controller.cc
M be/src/scheduling/admission-controller.h
M tests/custom_cluster/test_executor_groups.py
4 files changed, 167 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/65/24965/1
--
To view, visit http://gerrit.cloudera.org:8080/24965
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I84024a8d11911b639679029e34445ca52902ab93
Gerrit-Change-Number: 24965
Gerrit-PatchSet: 1
Gerrit-Owner: Anubhav Jindal <[email protected]>

Reply via email to