From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id D1E7D21060 for ; Mon, 8 Jul 2019 23:47:14 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id bxfe09ANGLDc for ; Mon, 8 Jul 2019 23:47:14 -0400 (EDT) Received: from smtpng3.m.smailru.net (smtpng3.m.smailru.net [94.100.177.149]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id 8B12521042 for ; Mon, 8 Jul 2019 23:47:14 -0400 (EDT) Date: Tue, 9 Jul 2019 06:46:50 +0300 From: Alexander Turenko Subject: [tarantool-patches] Re: [RFC 2/2] box/lua/console: Add support for lua output format Message-ID: <20190709034648.ngux4oxovmufsbhr@tkn_work_nb> References: <20190705210959.8527-1-gorcunov@gmail.com> <20190705210959.8527-3-gorcunov@gmail.com> <20190705225908.GC30966@atlas> <20190706065002.GB4602@uranus.lan> <20190706160239.GA21719@atlas> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20190706160239.GA21719@atlas> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: Konstantin Osipov Cc: Cyrill Gorcunov , tml On Sat, Jul 06, 2019 at 07:02:39PM +0300, Konstantin Osipov wrote: > * Cyrill Gorcunov [19/07/06 17:40]: > > On Sat, Jul 06, 2019 at 01:59:08AM +0300, Konstantin Osipov wrote: > > > * Cyrill Gorcunov [19/07/06 00:13]: > > > > > > LGTM. @totktonada, please provide a more thorough review. Okay. > > > > > > One question is what happens to the output on client reconnect. > > > > Since it gonna be new session the settings will be reset. I thought > > also that we might need to provide some box.cgf option for default > > output mode? > > Yes, but it could be box.session setting, not box.cfg. Related questions I'm tentative about: * Is there a way to keep an old session after reconnection? * Is there way to share a session for a connection pool? > > > > > Perhaps we should cache it on the client as well (this is the same > > > issue with console language). > > > > Well, seems so, still it is a bit unclear for me where to keep this > > cached value. > > I think having a default is good enough. > > > As to "default lua" for 2.x series we need to think how to tune > > up our test engine to setup yaml mode for all tests, since currently > > test results are in yaml and we definitely don't wanna update every > > test exsiting. > > Well, at least every new test should use the new output. There > aren't that many results, and it's just results - not tests - so > they won't be hard to merge (re-run the test and compare the > diff). Besides, 1.10 is more or less frozen, it accepts only minor > bugfixes. Now it's a perfect time. I guess the lua output format support will not be part of 1.10 branch? If we'll change all test result files, then cherry-picking anything backward to 1.10 will require extra work. So I don't think we should convert old result files. 1.10 is alive, so I doubt it actually can be frozen. Re using lua output for new test result files. We'll need to write an output format into a result file header. This requires to bump a result file format version and implement the new logic in test-run only for them (I don't see other backward compatible ways for now). I'm also afraid that the inconsistency in result file formats (one of which is not supported by 1.10) can give us problems if we'll decide to backport something to 1.10 after it will land to master with a test in the new format. So lua output by default for new tests seems to be doable feature, but will increase complexity of our test harness and will possibly problematic in some (rare?) cases. Are there any real problems with yaml result files?