From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <sergepetrenko@tarantool.org>
Received: from smtp53.i.mail.ru (smtp53.i.mail.ru [94.100.177.113])
 (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 C376045C304
 for <tarantool-patches@dev.tarantool.org>;
 Fri, 11 Dec 2020 10:15:10 +0300 (MSK)
References: <cover.1607633488.git.sergepetrenko@tarantool.org>
From: Serge Petrenko <sergepetrenko@tarantool.org>
Message-ID: <1748276e-801a-4e98-0289-9643c90e619f@tarantool.org>
Date: Fri, 11 Dec 2020 10:15:07 +0300
MIME-Version: 1.0
In-Reply-To: <cover.1607633488.git.sergepetrenko@tarantool.org>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Content-Language: en-GB
Subject: Re: [Tarantool-patches] [PATCH 0/4] make clear_synchro_queue commit
	everything
List-Id: Tarantool development patches <tarantool-patches.dev.tarantool.org>
List-Unsubscribe: <https://lists.tarantool.org/mailman/options/tarantool-patches>, 
 <mailto:tarantool-patches-request@dev.tarantool.org?subject=unsubscribe>
List-Archive: <https://lists.tarantool.org/pipermail/tarantool-patches/>
List-Post: <mailto:tarantool-patches@dev.tarantool.org>
List-Help: <mailto:tarantool-patches-request@dev.tarantool.org?subject=help>
List-Subscribe: <https://lists.tarantool.org/mailman/listinfo/tarantool-patches>, 
 <mailto:tarantool-patches-request@dev.tarantool.org?subject=subscribe>
To: Cyrill Gorcunov <gorcunov@gmail.com>
Cc: tarantool-patches@dev.tarantool.org

cc Cyrill Gorcunov.

Sorry, Cyrill, I misspelled your address.


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
>
-- 
Serge Petrenko