Matthew Jacobs has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/3272 Reviewed-by: Dan Hecht <[email protected]> Tested-by: Matthew Jacobs <[email protected]> --- M tests/custom_cluster/test_admission_controller.py 1 file changed, 30 insertions(+), 2 deletions(-) Approvals: Matthew Jacobs: Verified Dan Hecht: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/3272 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6682b15a5caac5824384c4b48a7b40afa2548954 Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]>
