From: Sergey Petrenko via Tarantool-patches <tarantool-patches@dev.tarantool.org>
To: Vladislav Shpilevoy <v.shpilevoy@tarantool.org>,
tarantool-patches@dev.tarantool.org, gorcunov@gmail.com
Subject: Re: [Tarantool-patches] [PATCH v2 0/5] Bootstrap voter
Date: Mon, 19 Jul 2021 16:27:09 +0200 [thread overview]
Message-ID: <fe3a9b33-aad4-9a0c-8c75-deb53d186630@tarantool.org> (raw)
In-Reply-To: <cover.1626627097.git.v.shpilevoy@tarantool.org>
18.07.2021 18:53, Vladislav Shpilevoy пишет:
> Changes in v2:
> - Moved 'manual' election mode fix into a separate commit;
> - Replaced raft_start/stop_candidate() with raft_promote/restore();
> - Renamed ballot.can_be_leader to ballot.can_lead.
>
> Branch: http://github.com/tarantool/tarantool/tree/gerold103/gh-6018-boot-voter
> Issue: https://github.com/tarantool/tarantool/issues/6018
>
> Vladislav Shpilevoy (5):
> replication: introduce ballot.can_lead
> box: save box_raft() into a variable
> raft: replace raft_start_candidate with _promote
> election: during bootstrap prefer candidates
> election: promote 'manual' bootstrap master
>
> .../unreleased/gh-6018-election-boot-voter.md | 4 +
> src/box/box.cc | 74 ++++++-----
> src/box/errcode.h | 2 +-
> src/box/iproto_constants.h | 1 +
> src/box/raft.c | 58 +++++++--
> src/box/raft.h | 4 +-
> src/box/replication.cc | 11 +-
> src/box/xrow.c | 14 ++-
> src/box/xrow.h | 2 +
> src/lib/raft/raft.c | 98 ++++++++-------
> src/lib/raft/raft.h | 13 +-
> test/box/error.result | 2 +-
> .../gh-6018-election-boot-voter.result | 116 ++++++++++++++++++
> .../gh-6018-election-boot-voter.test.lua | 59 +++++++++
> test/replication/gh-6018-master.lua | 17 +++
> test/replication/gh-6018-replica.lua | 15 +++
> test/replication/suite.cfg | 1 +
> test/unit/raft.c | 41 ++++---
> test/unit/raft.result | 22 ++--
> test/unit/raft_test_utils.c | 17 +--
> test/unit/raft_test_utils.h | 16 +--
> 21 files changed, 438 insertions(+), 149 deletions(-)
> create mode 100644 changelogs/unreleased/gh-6018-election-boot-voter.md
> create mode 100644 test/replication/gh-6018-election-boot-voter.result
> create mode 100644 test/replication/gh-6018-election-boot-voter.test.lua
> create mode 100644 test/replication/gh-6018-master.lua
> create mode 100644 test/replication/gh-6018-replica.lua
>
Thanks for the fixes! LGTM.
next prev parent reply other threads:[~2021-07-19 14:27 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-18 16:53 Vladislav Shpilevoy via Tarantool-patches
2021-07-18 16:53 ` [Tarantool-patches] [PATCH v2 1/5] replication: introduce ballot.can_lead Vladislav Shpilevoy via Tarantool-patches
2021-07-21 21:38 ` Vladislav Shpilevoy via Tarantool-patches
2021-07-18 16:53 ` [Tarantool-patches] [PATCH v2 2/5] box: save box_raft() into a variable Vladislav Shpilevoy via Tarantool-patches
2021-07-18 16:53 ` [Tarantool-patches] [PATCH v2 3/5] raft: replace raft_start_candidate with _promote Vladislav Shpilevoy via Tarantool-patches
2021-07-18 16:53 ` [Tarantool-patches] [PATCH v2 4/5] election: during bootstrap prefer candidates Vladislav Shpilevoy via Tarantool-patches
2021-07-18 16:53 ` [Tarantool-patches] [PATCH v2 5/5] election: promote 'manual' bootstrap master Vladislav Shpilevoy via Tarantool-patches
2021-07-19 14:27 ` Sergey Petrenko via Tarantool-patches [this message]
2021-07-20 8:18 ` [Tarantool-patches] [PATCH v2 0/5] Bootstrap voter Cyrill Gorcunov via Tarantool-patches
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fe3a9b33-aad4-9a0c-8c75-deb53d186630@tarantool.org \
--to=tarantool-patches@dev.tarantool.org \
--cc=gorcunov@gmail.com \
--cc=sergepetrenko@tarantool.org \
--cc=v.shpilevoy@tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH v2 0/5] Bootstrap voter' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox