You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Matthew Jacobs (Code Review)" <ge...@cloudera.org> on 2016/06/01 19:02:20 UTC
[Impala-CR](cdh5-trunk) IMPALA-3600: Add missing admission control tests
Matthew Jacobs has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3272
Change subject: IMPALA-3600: Add missing admission control tests
......................................................................
IMPALA-3600: Add missing admission control tests
* -require_username (not strictly admission control related
but it came up in the context of RM).
* Coverage of failure cases: The handling of the full queue
case wasn't being verified. This changes existing stress
test to expect a specific message when the queue is full.
* Requesting MAXINT memory, which previously led to an
overflow in the pool-level mem tracker accounting.
This does not yet address:
* Changing pool cfg while running
* Verify profile string for queued reason
This is just a minimal incremental change to get additional
coverage. Right now, many of the tests rely on some
pre-defined configuration files which is cumbersome. In the
future, we plan on refreshing the configuration story at
which point we should also build more general test
infrastructure for easily testing different configurations.
Change-Id: I6682b15a5caac5824384c4b48a7b40afa2548954
---
M tests/custom_cluster/test_admission_controller.py
1 file changed, 30 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/72/3272/1
--
To view, visit http://gerrit.cloudera.org:8080/3272
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6682b15a5caac5824384c4b48a7b40afa2548954
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>