From: Vladislav Shpilevoy <v.shpilevoy@tarantool.org> To: Alex Khatskevich <avkhatskevich@tarantool.org>, tarantool-patches@freelists.org Subject: [tarantool-patches] Re: [PATCH][vshard] Reload reloadable fiber Date: Wed, 27 Jun 2018 14:53:57 +0300 [thread overview] Message-ID: <4dc68f8e-ad09-d818-3ea2-2eb6cb890d37@tarantool.org> (raw) In-Reply-To: <5536a4f7-f598-a7d4-0b92-a3483a7a339b@tarantool.org> On 27/06/2018 14:45, Alex Khatskevich wrote: > I have rebased and pushed, but... > > Non of the tests fails. > > As far as I know, there is a couple of blinking tests in vshard. VShard now has no unstable tests. I got error on your new test, and even with no rebase it appears sometimes: ====================================================================================== WORKR TEST PARAMS RESULT --------------------------------------------------------------------------------- [001] unit/util.test.lua [ fail ] [001] [001] Test failed! Result content mismatch: [001] --- unit/util.result Wed Jun 27 14:46:34 2018 [001] +++ unit/util.reject Wed Jun 27 14:52:17 2018 [001] @@ -39,11 +39,11 @@ [001] ... [001] test_run:grep_log('default', 'Worker_name: reloadable function reloadable_function has been changed') [001] --- [001] -- 'Worker_name: reloadable function reloadable_function has been changed' [001] +- null [001] ... [001] test_run:grep_log('default', 'Worker_name is reloaded, restarting') [001] --- [001] -- Worker_name is reloaded, restarting [001] +- null [001] ... [001] test_run:grep_log('default', 'Worker_name has been started') [001] --- [001] I think it is because you cancel fiber too early in the test. Please, repair. > > > On 27.06.2018 14:34, Vladislav Shpilevoy wrote: >> Please, rebase on the latest master branch. When I do, >> the tests fail.
next prev parent reply other threads:[~2018-06-27 11:53 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-06-14 11:42 [tarantool-patches] " AKhatskevich 2018-06-14 11:58 ` AKhatskevich 2018-06-21 12:04 ` [tarantool-patches] " Vladislav Shpilevoy 2018-06-26 12:43 ` Alex Khatskevich 2018-06-26 12:50 ` Alex Khatskevich 2018-06-27 9:54 ` Vladislav Shpilevoy 2018-06-27 10:44 ` Alex Khatskevich 2018-06-27 11:34 ` Vladislav Shpilevoy 2018-06-27 11:45 ` Alex Khatskevich 2018-06-27 11:53 ` Vladislav Shpilevoy [this message] 2018-06-27 13:11 ` Vladislav Shpilevoy
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4dc68f8e-ad09-d818-3ea2-2eb6cb890d37@tarantool.org \ --to=v.shpilevoy@tarantool.org \ --cc=avkhatskevich@tarantool.org \ --cc=tarantool-patches@freelists.org \ --subject='[tarantool-patches] Re: [PATCH][vshard] Reload reloadable fiber' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox