From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 7 Aug 2018 19:50:31 +0300 From: Vladimir Davydov Subject: Re: [tarantool-patches] [PATCH] replication: fix a failing assert in replica_on_applier_disconnect() Message-ID: <20180807165031.e2avj7waurok3zpk@esperanza> References: <20180803055905.69108-1-sergepetrenko@tarantool.org> <20180803155745.tmndjr52n6igtdno@tarantool.org> <81999702-C603-423E-92C9-199CE605FED4@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <81999702-C603-423E-92C9-199CE605FED4@tarantool.org> To: Sergey Petrenko Cc: tarantool-patches@freelists.org, Kirill Yukhin List-ID: On Mon, Aug 06, 2018 at 05:14:05PM +0300, Sergey Petrenko wrote: > diff --git a/test/replication/misc.test.lua b/test/replication/misc.test.lua > index c05e52165..27c1a4821 100644 > --- a/test/replication/misc.test.lua > +++ b/test/replication/misc.test.lua > @@ -81,3 +81,15 @@ test_run:cmd("switch default") > test_run:drop_cluster(SERVERS) > > box.schema.user.revoke('guest', 'replication') > + > +-- gh-3510 assertion failure in replica_on_applier_disconnect() > +test_run:cmd('create server er_load1 with script="replication/er_load1.lua"') > +test_run:cmd('create server er_load2 with script="replication/er_load2.lua"') > +test_run:cmd('start server er_load1 with wait=False, wait_load=False') > +test_run:cmd('start server er_load2 with wait=False, wait_load=False') > +require('fiber').sleep(0.5) > +test_run:cmd('stop server er_load1') > +require('fiber').sleep(1) These sleep()'s are way too long. They will make our tests run forever. Please change to 0.00001 or smaller or rewrite the test so that you do not need them.