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>

Reply via email to