Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22787 )
Change subject: IMPALA-13971: Deflake TestAdmissionController.test_user_loads_rules ...................................................................... IMPALA-13971: Deflake TestAdmissionController.test_user_loads_rules TestAdmissionController.test_user_loads_rules is flaky for not failing the last query that should exceed the user quota. The test executes queries in a round-robin fashion across all impalad. These impalads are expected to synchronize user quota count through statestore updates. This patch attempts to deflake the test by raising the heartbeat wait time from 1 heartbeat period to 3 hearbeat periods. It also changes the reject query to a fast version of SLOW_QUERY (without sleep) so the test can fail fast if it is not rejected. Testing: Loop the test 50 times and pass them all. Change-Id: Ib2ae8e1c2edf174edbf0e351d3c2ed06a0539f08 Reviewed-on: http://gerrit.cloudera.org:8080/22787 Reviewed-by: Riza Suminto <riza.sumi...@cloudera.com> Tested-by: Riza Suminto <riza.sumi...@cloudera.com> --- M tests/custom_cluster/test_admission_controller.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Riza Suminto: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22787 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib2ae8e1c2edf174edbf0e351d3c2ed06a0539f08 Gerrit-Change-Number: 22787 Gerrit-PatchSet: 6 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
