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 <[email protected]>
