Wenzhe Zhou has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20699 )
Change subject: IMPALA-12556: Fix flaky test test_two_statestored_with_force_active ...................................................................... IMPALA-12556: Fix flaky test test_two_statestored_with_force_active Test test_two_statestored_with_force_active failed occasionally by cause of both statestore instances assigned with active roles. This patch fixes the issue to handle the case that both statestore instances are restarted with flag "statestore_force_active" in the same way as both statestore instances are restarted without flag "statestore_force_active". Testing: - Repeatedly ran test_two_statestored_with_force_active on Jenkins for hundreds of times without failure. - Repeatedly ran test_two_statestored_with_force_active on local machine for thousand times without failure. - Repeatedly ran all tests in test_statestored_ha.py for over 12 hours on Jenkins without failure. - Passed core tests. Change-Id: I3e6f85233ff6fa747a6aa5ef8d093627885d20b2 Reviewed-on: http://gerrit.cloudera.org:8080/20699 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Wenzhe Zhou <wz...@cloudera.com> --- M be/src/statestore/statestore-subscriber.cc M be/src/statestore/statestore.cc 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved Wenzhe Zhou: Verified -- To view, visit http://gerrit.cloudera.org:8080/20699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3e6f85233ff6fa747a6aa5ef8d093627885d20b2 Gerrit-Change-Number: 20699 Gerrit-PatchSet: 4 Gerrit-Owner: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>