CRZbulabula commented on PR #1475: URL: https://github.com/apache/ratis/pull/1475#issuecomment-4585944423
Pushed `fb3ee4ee` to address the latest CI failures. Summary: - `TestManualRestoreSnapshot` now verifies that the restarted follower has restored and applied at least the saved snapshot index, instead of racing against the leader current lastApplied value. - `testAddServerForWaitReady` now waits until the new server impls report running after the ADD configuration change, and always cleans up the START_COMPLETE injection in a finally block. Local verification: - `mvn -pl ratis-examples -am -Dtest=TestManualRestoreSnapshot test` - `mvn -pl ratis-test -am -Dtest=TestLeaderElectionWithSimulatedRpc#testAddServerForWaitReady,TestLeaderElectionWithGrpc#testAddServerForWaitReady,TestLeaderElectionWithNetty#testAddServerForWaitReady test` GitHub Actions for `fb3ee4ee` are green; coverage is skipped as expected. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
