Skip to content
Snippets Groups Projects

Refactor join loop (v2)

Merged Yaroslav Dynnikov requested to merge refactor-join-loop-2 into master
All threads resolved!

This PR is completely identical to !156 (closed), but is rebased on top of !157 (merged).

Edited by Yaroslav Dynnikov

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Georgy Moshkin
  • Yaroslav Dynnikov mentioned in merge request !156 (closed)

    mentioned in merge request !156 (closed)

  • added 8 commits

    • c5c0ef27...803d2a9a - 2 commits from branch master
    • 3764f758 - chore: replace another "if" with another "match"
    • 3bd7547f - refactor: get rid of join_loop
    • fc4201bb - chore: rename back active -> is_active
    • 494d4e75 - fix: notify latch when handling ConfChange V1
    • 986b9469 - fix: inner test_storage_peers
    • aaa8b9a2 - fix: remove old peer when putting new one

    Compare with previous version

  • added 2 commits

    • 8fde0ba8 - fix(test): fix topology unit tests
    • 72787c37 - fix: conftest cluster deploying

    Compare with previous version

  • added 2 commits

    • ba36dd62 - refactor: use Health instead of bool
    • b7049dd1 - revise two more pytests

    Compare with previous version

  • Yaroslav Dynnikov added 12 commits

    added 12 commits

    • ce08ab98 - refactor: extract init_common function
    • e9f48c4e - refactor: use RaftId for raft_id
    • 8800b5c0 - refactor: check raft.state directly
    • 0a1c150f - refactor: use events instead of conds
    • c693710f - chore: fix warnings
    • 46567ebc - feat(switchover): implement voter switchover rules
    • 8401d4fb - fix: revise test_joining
    • c991e1db - fix: cluster deployment
    • ad6f79ad - fix: ignore raft messages with wrong destination
    • 169c02fe - chore: remove obsolete code snippet
    • a16b2cc4 - fix: avoid dropping msg_timeout_now
    • a4ace03e - test: remove obsolete tests

    Compare with previous version

  • added 2 commits

    • fa3b5d16 - fix: avoid dropping msg_timeout_now
    • 353b3675 - test: remove obsolete tests

    Compare with previous version

  • Georgy Moshkin approved this merge request

    approved this merge request

  • Yaroslav Dynnikov marked this merge request as ready

    marked this merge request as ready

  • Yaroslav Dynnikov resolved all threads

    resolved all threads

  • Please register or sign in to reply
    Loading