From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id 7F0746EC55; Tue, 27 Jul 2021 13:06:19 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 7F0746EC55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1627380379; bh=gHEpUvhUXLH33o5j3Wm9VMh/Kvh5/6MKDTS7MlYXdSA=; h=Date:To:Cc:References:In-Reply-To:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=TsG/wPXdmqEm3sj+sLSBATHHHj9MtfNy2M4iX9qSQzK0dAPtX7103lsNGncW1rIND 5k9ErAVf4U/RMV2PatrYTD/teqORfdY7wpYxSJZmKmsQAAyx06t5+s8gZKxanNwkeu 2OyA2DOgz+/hb9x1NfqfUfxrWhFGbcV3ZB5KyDv4= Received: from smtpng2.i.mail.ru (smtpng2.i.mail.ru [94.100.179.3]) (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 52F916EC55 for ; Tue, 27 Jul 2021 13:06:18 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 52F916EC55 Received: by smtpng2.m.smailru.net with esmtpa (envelope-from ) id 1m8Jyb-00044b-Jg; Tue, 27 Jul 2021 13:06:18 +0300 Date: Tue, 27 Jul 2021 13:06:16 +0300 To: Vladislav Shpilevoy Cc: tarantool-patches@dev.tarantool.org Message-ID: <20210727100616.qjnhzfiggwqo2uag@esperanza> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-7564579A: B8F34718100C35BD X-77F55803: 4F1203BC0FB41BD941C43E597735A9C351B198F4576AC7B2770D7874BA03B4AE182A05F5380850405E85A482E5B67D7251014C20F154DB5A6C0CD884B64CFA97A0F075CD11571492 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7A33E1178EA603666EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637E8DF562B6063DE0A8638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D880AF3B7D8BE18B9D7DE5C2E08C576C95117882F4460429724CE54428C33FAD305F5C1EE8F4F765FCAA867293B0326636D2E47CDBA5A96583BD4B6F7A4D31EC0BC014FD901B82EE079FA2833FD35BB23D27C277FBC8AE2E8BAA867293B0326636D2E47CDBA5A96583BA9C0B312567BB231DD303D21008E29813377AFFFEAFD269A417C69337E82CC2E827F84554CEF50127C277FBC8AE2E8BA83251EDC214901ED5E8D9A59859A8B6753C3A5E0A5AB5B7089D37D7C0E48F6C5571747095F342E88FB05168BE4CE3AF X-C1DE0DAB: 0D63561A33F958A5D05BF2E9FB9A589330B6BEF1195C2208103CDA927135587FD59269BC5F550898D99A6476B3ADF6B47008B74DF8BB9EF7333BD3B22AA88B938A852937E12ACA75FA7FF33AA1A4D21C410CA545F18667F91A7EA1CDA0B5A7A0 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D34F6CC1FD336696336EB6853BB685CAC6C7711AC0DDBCED2B6F6AB57181EC9EB8673B0437A3EFB4A101D7E09C32AA3244CBBEBE666B057A4627C9DE444FB05079C7C0C08F7987826B9FACE5A9C96DEB163 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojMEANdStWW5/z4lH+nIZMKQ== X-Mailru-Sender: 689FA8AB762F7393C37E3C1AEC41BA5D647001623ED4BC0341B2E89CC01FD855274CEFED1673C562683ABF942079399BFB559BB5D741EB966A65DFF43FF7BE03240331F90058701C67EA787935ED9F1B X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH] net.box: drop conn:timeout() X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Vladimir Davydov via Tarantool-patches Reply-To: Vladimir Davydov Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" On Tue, Jul 27, 2021 at 12:37:17AM +0200, Vladislav Shpilevoy wrote: > See 5 comments below. All fixed, thanks. The patch is below: https://github.com/tarantool/tarantool/tree/vdavydov/net-box-drop-conn-timeout >From f1dbd33e4ccbc586d12460b9061c4c6a30b95f32 Mon Sep 17 00:00:00 2001 From: Vladimir Davydov Date: Mon, 26 Jul 2021 11:59:15 +0300 Subject: [PATCH] net.box: drop conn:timeout() conn:timeout(timeout) writes (now + timeout) to conn._deadlines table keyed by fiber.self(). The deadline will then be used by all conn methods unless specified explicitly in the options argument. Usage of fiber.self() degrades net.box performance by about 15% in case timeout is not specified, because it is a C call and so interferes with JIT. See the ticket for the test details. Since the feature is unusable and was deprecated more than four years ago (in 1.7.4) in favor of specifying timeout in the options argument (see commit 58da206c33197 ("net.box: add { timeout = } option to requests")), it should be fine to drop it. Closes #6242 @TarantoolBot document Title: Delete documentation entry for net.box conn:timeout() conn:timeout() was deprecated more than four years ago (in 1.7.4). Since it results in performance degradation in case timeout is not specified explicitly in request options, we decided to drop it. Please remove it from the documentation as well: https://www.tarantool.io/en/doc/latest/reference/reference_lua/net_box/#conn-timeout diff --git a/changelogs/unreleased/gh-6242-net-box-drop-conn-timeout.md b/changelogs/unreleased/gh-6242-net-box-drop-conn-timeout.md new file mode 100644 index 000000000000..9571c0afc577 --- /dev/null +++ b/changelogs/unreleased/gh-6242-net-box-drop-conn-timeout.md @@ -0,0 +1,9 @@ +## feature/core + +* **[Breaking change]** timeout() method of net.box connection, which was + marked deprecated more than four years ago (in 1.7.4) was dropped, because + it negatively affected performance of hot net.box methods, like call() and + select(), in case those are called without specifying a timeout (gh-6242). + +--- +Breaking change: timeout() method of net.box connection was dropped. diff --git a/src/box/lua/net_box.lua b/src/box/lua/net_box.lua index 3878abf21914..d70927989b71 100644 --- a/src/box/lua/net_box.lua +++ b/src/box/lua/net_box.lua @@ -1047,9 +1047,6 @@ local function new_sm(host, port, opts, connection, greeting) setmetatable(remote, console_mt) else setmetatable(remote, remote_mt) - -- @deprecated since 1.7.4 - remote._deadlines = setmetatable({}, {__mode = 'k'}) - remote._space_mt = space_metatable(remote) remote._index_mt = index_metatable(remote) if opts.call_16 then @@ -1176,17 +1173,11 @@ function remote_methods:_request(method, opts, request_ctx, ...) return res end if opts.timeout then - -- conn.space:request(, { timeout = timeout }) deadline = fiber_clock() + opts.timeout - else - -- conn:timeout(timeout).space:request() - -- @deprecated since 1.7.4 - deadline = self._deadlines[fiber_self()] end on_push = opts.on_push or on_push_sync_default on_push_ctx = opts.on_push_ctx else - deadline = self._deadlines[fiber_self()] on_push = on_push_sync_default end -- Execute synchronous request. @@ -1289,28 +1280,9 @@ end function remote_methods:wait_state(state, timeout) check_remote_arg(self, 'wait_state') - if timeout == nil then - local deadline = self._deadlines[fiber_self()] - timeout = deadline and max(0, deadline - fiber_clock()) - end return self._transport.wait_state(state, timeout) end -local compat_warning_said = false - --- @deprecated since 1.7.4 -function remote_methods:timeout(timeout) - check_remote_arg(self, 'timeout') - if not compat_warning_said then - compat_warning_said = true - log.warn("netbox:timeout(timeout) is deprecated since 1.7.4, ".. - "please use space:(..., {timeout = t}) instead.") - end - -- Sic: this is broken by design - self._deadlines[fiber_self()] = (timeout and fiber_clock() + timeout) - return self -end - function remote_methods:_install_schema(schema_version, spaces, indices, collations) local sl, space_mt, index_mt = {}, self._space_mt, self._index_mt diff --git a/test/box/net.box_count_inconsistent_gh-3262.result b/test/box/net.box_count_inconsistent_gh-3262.result index c77677625d97..286f9ded8aea 100644 --- a/test/box/net.box_count_inconsistent_gh-3262.result +++ b/test/box/net.box_count_inconsistent_gh-3262.result @@ -427,55 +427,6 @@ cn:eval('return ret_after(...)', {1.00}, { timeout = 1e-9 }) --- - error: Timeout exceeded ... --- --- :timeout() --- @deprecated since 1.7.4 --- -cn:timeout(1).space.net_box_test_space.index.primary:select{234} ---- -- - [234, 1, 2, 3] -... -cn:call('ret_after', {.01}) ---- -- [[0.01]] -... -cn:timeout(1):call('ret_after', {.01}) ---- -- [[0.01]] -... -cn:timeout(.01):call('ret_after', {1}) ---- -- error: Timeout exceeded -... -cn = remote:timeout(0.0000000001):connect(LISTEN.host, LISTEN.service, { user = 'netbox', password = '123' }) ---- -... -cn:close() ---- -... -cn = remote:timeout(1):connect(LISTEN.host, LISTEN.service, { user = 'netbox', password = '123' }) ---- -... -remote.self:ping() ---- -- true -... -remote.self.space.net_box_test_space:select{234} ---- -- - [234, 1, 2, 3] -... -remote.self:timeout(123).space.net_box_test_space:select{234} ---- -- - [234, 1, 2, 3] -... -remote.self:is_connected() ---- -- true -... -remote.self:wait_connected() ---- -- true -... cn:close() --- ... diff --git a/test/box/net.box_count_inconsistent_gh-3262.test.lua b/test/box/net.box_count_inconsistent_gh-3262.test.lua index e84e85cf3829..a6d87cdcc830 100644 --- a/test/box/net.box_count_inconsistent_gh-3262.test.lua +++ b/test/box/net.box_count_inconsistent_gh-3262.test.lua @@ -155,26 +155,6 @@ cn:call('ret_after', {1.00}, { timeout = 1e-9 }) cn:eval('return ret_after(...)', {0.01}, { timeout = 1.00 }) cn:eval('return ret_after(...)', {1.00}, { timeout = 1e-9 }) --- --- :timeout() --- @deprecated since 1.7.4 --- - -cn:timeout(1).space.net_box_test_space.index.primary:select{234} -cn:call('ret_after', {.01}) -cn:timeout(1):call('ret_after', {.01}) -cn:timeout(.01):call('ret_after', {1}) - -cn = remote:timeout(0.0000000001):connect(LISTEN.host, LISTEN.service, { user = 'netbox', password = '123' }) -cn:close() -cn = remote:timeout(1):connect(LISTEN.host, LISTEN.service, { user = 'netbox', password = '123' }) - -remote.self:ping() -remote.self.space.net_box_test_space:select{234} -remote.self:timeout(123).space.net_box_test_space:select{234} -remote.self:is_connected() -remote.self:wait_connected() - cn:close() -- cleanup database after tests space:drop()