From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp60.i.mail.ru (smtp60.i.mail.ru [217.69.128.40]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id AE49445C304 for ; Fri, 11 Dec 2020 12:19:57 +0300 (MSK) References: From: Serge Petrenko Message-ID: <696df4e5-d79a-443c-fb2b-9710aecbd928@tarantool.org> Date: Fri, 11 Dec 2020 12:19:56 +0300 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8"; format="flowed" Content-Transfer-Encoding: 8bit Content-Language: ru Subject: Re: [Tarantool-patches] [PATCH 0/4] make clear_synchro_queue commit everything List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: v.shpilevoy@tarantool.org, Cyrill Gorcunov Cc: tarantool-patches@dev.tarantool.org 10.12.2020 23:55, Serge Petrenko пишет: > The patchset fails `replication/election_qsync_stress.test.lua` test. > I haven't found the reason for this yet, but I still think it may be reviewed > while I'm trying to figure out where the test failures come from. > > Serge Petrenko (4): > box: add a single execution guard to clear_synchro_queue > relay: rename is_raft_enabled message to relay_is_running > relay: introduce relay_lsn_watcher > box: rework clear_synchro_queue to commit everything > > src/box/box.cc | 143 +++++++++++++++--- > src/box/box.h | 2 +- > src/box/lua/ctl.c | 4 +- > src/box/relay.cc | 120 +++++++++++---- > src/box/relay.h | 44 ++++++ > test/replication/election_replica.lua | 5 +- > ...5435-clear-synchro-queue-commit-all.result | 137 +++++++++++++++++ > ...35-clear-synchro-queue-commit-all.test.lua | 60 ++++++++ > test/replication/suite.cfg | 1 + > 9 files changed, 460 insertions(+), 56 deletions(-) > create mode 100644 test/replication/gh-5435-clear-synchro-queue-commit-all.result > create mode 100644 test/replication/gh-5435-clear-synchro-queue-commit-all.test.lua > https://github.com/tarantool/tarantool/issues/5435 sp/gh-5435-new-leader-commit-all -- Serge Petrenko