From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf1-f46.google.com (mail-lf1-f46.google.com [209.85.167.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 3D4F74765E0 for ; Tue, 22 Dec 2020 17:32:44 +0300 (MSK) Received: by mail-lf1-f46.google.com with SMTP id o19so32515497lfo.1 for ; Tue, 22 Dec 2020 06:32:44 -0800 (PST) Date: Tue, 22 Dec 2020 17:32:40 +0300 From: Cyrill Gorcunov Message-ID: <20201222143240.GB8261@grain> References: <20201222111408.48368-1-gorcunov@gmail.com> <8b570db3-858d-4475-1f9f-7e5c84321129@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8b570db3-858d-4475-1f9f-7e5c84321129@tarantool.org> Subject: Re: [Tarantool-patches] [PATCH v6 0/5] qsync: evaluate replication_synchro_quorum dynamically List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vladislav Shpilevoy Cc: tml On Tue, Dec 22, 2020 at 02:49:06PM +0100, Vladislav Shpilevoy wrote: > Hi! Thanks for the patchset! > > Looks good, except that the test is still flaky. I run it > with the same command as earlier, and got this: Vlad, I've increased timeout for console buffer flushing. Could you please re-fetch the branch (I've force pushed it) and test out? The timeout trick doesn't look 100% reliable and I think I will need to enhance the test to use new qsync statistics (which is not yet done). Thus I'll make a linked issue for this and once qsync stat is out we will fix the test for sure.