Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10845 )
Change subject: IMPALA-7185: low statestore custom cluster interval ...................................................................... Patch Set 2: (1 comment) Thanks for putting this together. http://gerrit.cloudera.org:8080/#/c/10845/2/tests/custom_cluster/test_admission_controller.py File tests/custom_cluster/test_admission_controller.py: http://gerrit.cloudera.org:8080/#/c/10845/2/tests/custom_cluster/test_admission_controller.py@287 PS2, Line 287: "-default_query_options=mem_limit=200000000")) Most of the tests in this file use _STATESTORED_ARGS (all but test_mem_limit_upper_bound prior to this change, and that one might not be intentional). Is there something unique about this test that lets you remove it here? (Or can we remove it for other tests here?) -- To view, visit http://gerrit.cloudera.org:8080/10845 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia5d1612283ff420d95b0dd0ca5a2a67f56765f79 Gerrit-Change-Number: 10845 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Comment-Date: Fri, 29 Jun 2018 23:24:13 +0000 Gerrit-HasComments: Yes
