[Tarantool-patches] [PATCH v10 0/4] box/replication: add missing diag set and fix sigsegv

Cyrill Gorcunov gorcunov at gmail.com
Fri Feb 14 17:03:35 MSK 2020


In v10 I tried to sanitize code a bit. Still there are few problems we will
need (well, I'll need) to solve:

 - move error injection deeper into transaction code
 - keep former error
 - make sure that we really need separated replicaset diag instance
   (i must confess i still not follow why we need it)

Anyway with the patch we stop causing nil dereference and write
error log. Same time since I'll have to deal with inmemory wal
task I won't leave these FIXME and continue working on them.

Cyrill Gorcunov (4):
  box/request: add missing OutOfMemory diag_set
  box/applier: add missing diag_set on region_alloc failure
  box/applier: prevent nil dereference on applier rollback
  test: add replication/applier-rollback

 src/box/applier.cc                          |  43 +++++-
 src/box/request.c                           |   8 +-
 src/lib/core/errinj.h                       |   1 +
 test/box/errinj.result                      |   1 +
 test/replication/applier-rollback-slave.lua |  16 ++
 test/replication/applier-rollback.result    | 160 ++++++++++++++++++++
 test/replication/applier-rollback.test.lua  |  79 ++++++++++
 test/replication/suite.ini                  |   2 +-
 8 files changed, 305 insertions(+), 5 deletions(-)
 create mode 100644 test/replication/applier-rollback-slave.lua
 create mode 100644 test/replication/applier-rollback.result
 create mode 100644 test/replication/applier-rollback.test.lua


base-commit: 95b9a48daf00b6686b8e1993a023daa0755b460e
-- 
2.20.1



More information about the Tarantool-patches mailing list