[Tarantool-patches] [PATCH v3 00/12] forbid implicit limbo ownership transition
Serge Petrenko
sergepetrenko at tarantool.org
Tue Jun 29 01:12:46 MSK 2021
Changes in v3:
- change demote() behaviour as discussed with Vlad:
* make it work only on the current leader
* make it demote the current leader and always
bump the term
- change how limbo and raft snapshots are sent in response
to JOIN:
* encode replica's version in JOIN request
* introduce a special stage: JOIN_META with raft and limbo
snapshots. Send it based on replica's version.
https://github.com/tarantool/tarantool/issues/5438
https://github.com/tarantool/tarantool/issues/6034
https://github.com/tarantool/tarantool/tree/sp/gh-6034-empty-limbo-transition
Serge Petrenko (12):
replication: always send raft state to subscribers
txn_limbo: fix promote term filtering
raft: refactor raft_new_term()
box: make promote always bump the term
replication: forbid implicit limbo owner transition
box: allow calling promote on a candidate
box: introduce `box.ctl.demote`
txn_limbo: persist the latest effective promote in snapshot
replication: encode version in JOIN request
replication: add META stage to JOIN
replication: send latest effective promote in initial join
replication: send current Raft term in join response
src/box/applier.cc | 27 ++-
src/box/box.cc | 101 ++++++----
src/box/box.h | 3 +
src/box/errcode.h | 2 +
src/box/iproto_constants.h | 12 +-
src/box/lua/ctl.c | 9 +
src/box/lua/info.c | 4 +-
src/box/memtx_engine.c | 32 +++
src/box/raft.c | 36 ++++
src/box/raft.h | 4 +
src/box/relay.cc | 36 +++-
src/box/relay.h | 4 +-
src/box/txn_limbo.c | 90 ++++++---
src/box/txn_limbo.h | 14 ++
src/box/xrow.c | 4 +-
src/box/xrow.h | 25 ++-
src/lib/raft/raft.c | 3 +-
test/box/alter.result | 2 +-
test/box/error.result | 2 +
test/replication/election_basic.result | 3 +
test/replication/election_basic.test.lua | 1 +
test/replication/election_qsync.result | 3 +
test/replication/election_qsync.test.lua | 1 +
.../gh-4114-local-space-replication.result | 7 +-
.../gh-4114-local-space-replication.test.lua | 4 +-
.../gh-5140-qsync-casc-rollback.result | 6 +
.../gh-5140-qsync-casc-rollback.test.lua | 2 +
.../gh-5144-qsync-dup-confirm.result | 6 +
.../gh-5144-qsync-dup-confirm.test.lua | 2 +
.../gh-5163-qsync-restart-crash.result | 6 +
.../gh-5163-qsync-restart-crash.test.lua | 2 +
.../gh-5167-qsync-rollback-snap.result | 6 +
.../gh-5167-qsync-rollback-snap.test.lua | 2 +
.../gh-5195-qsync-replica-write.result | 10 +-
.../gh-5195-qsync-replica-write.test.lua | 6 +-
.../gh-5213-qsync-applier-order-3.result | 9 +
.../gh-5213-qsync-applier-order-3.test.lua | 3 +
.../gh-5213-qsync-applier-order.result | 6 +
.../gh-5213-qsync-applier-order.test.lua | 2 +
.../replication/gh-5288-qsync-recovery.result | 6 +
.../gh-5288-qsync-recovery.test.lua | 2 +
.../gh-5298-qsync-recovery-snap.result | 6 +
.../gh-5298-qsync-recovery-snap.test.lua | 2 +
.../gh-5426-election-on-off.result | 3 +
.../gh-5426-election-on-off.test.lua | 1 +
.../gh-5433-election-restart-recovery.result | 3 +
...gh-5433-election-restart-recovery.test.lua | 1 +
...sync-clear-synchro-queue-commit-all.result | 3 +
...nc-clear-synchro-queue-commit-all.test.lua | 1 +
test/replication/gh-5438-raft-state.result | 66 ++++++
test/replication/gh-5438-raft-state.test.lua | 29 +++
test/replication/gh-5440-qsync-ro.result | 133 ------------
test/replication/gh-5440-qsync-ro.test.lua | 53 -----
.../gh-5446-qsync-eval-quorum.result | 7 +
.../gh-5446-qsync-eval-quorum.test.lua | 3 +
.../gh-5506-election-on-off.result | 3 +
.../gh-5506-election-on-off.test.lua | 1 +
.../gh-5566-final-join-synchro.result | 6 +
.../gh-5566-final-join-synchro.test.lua | 2 +
.../gh-5874-qsync-txn-recovery.result | 6 +
.../gh-5874-qsync-txn-recovery.test.lua | 2 +
.../gh-6032-promote-wal-write.result | 3 +
.../gh-6032-promote-wal-write.test.lua | 1 +
.../gh-6034-limbo-ownership.result | 189 ++++++++++++++++++
.../gh-6034-limbo-ownership.test.lua | 69 +++++++
.../gh-6034-promote-bump-term.result | 40 ++++
.../gh-6034-promote-bump-term.test.lua | 17 ++
.../gh-6057-qsync-confirm-async-no-wal.result | 7 +
...h-6057-qsync-confirm-async-no-wal.test.lua | 3 +
test/replication/hang_on_synchro_fail.result | 6 +
.../replication/hang_on_synchro_fail.test.lua | 2 +
test/replication/qsync_advanced.result | 12 ++
test/replication/qsync_advanced.test.lua | 4 +
test/replication/qsync_basic.result | 33 ++-
test/replication/qsync_basic.test.lua | 16 +-
test/replication/qsync_errinj.result | 6 +
test/replication/qsync_errinj.test.lua | 2 +
test/replication/qsync_snapshots.result | 6 +
test/replication/qsync_snapshots.test.lua | 2 +
test/replication/qsync_with_anon.result | 6 +
test/replication/qsync_with_anon.test.lua | 2 +
test/replication/replica_rejoin.result | 77 ++++---
test/replication/replica_rejoin.test.lua | 50 ++---
test/replication/suite.cfg | 4 +-
test/unit/raft.c | 15 +-
test/unit/raft.result | 3 +-
86 files changed, 1039 insertions(+), 372 deletions(-)
create mode 100644 test/replication/gh-5438-raft-state.result
create mode 100644 test/replication/gh-5438-raft-state.test.lua
delete mode 100644 test/replication/gh-5440-qsync-ro.result
delete mode 100644 test/replication/gh-5440-qsync-ro.test.lua
create mode 100644 test/replication/gh-6034-limbo-ownership.result
create mode 100644 test/replication/gh-6034-limbo-ownership.test.lua
create mode 100644 test/replication/gh-6034-promote-bump-term.result
create mode 100644 test/replication/gh-6034-promote-bump-term.test.lua
--
2.30.1 (Apple Git-130)
More information about the Tarantool-patches
mailing list