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 82B386EC55; Sat, 31 Jul 2021 02:11:34 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 82B386EC55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1627686694; bh=dir0JMuzxDKxI7rADsce5IO3IFq98UIClEXlHMNIZV8=; h=To:Cc:References:In-Reply-To:Date:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=TdsYirVP3Bs8vgR0AFo0NO35cbhIe4uUFaIbKNeOj1HX1qjjTenLmFi2kU2BBg+Z+ gf30eUKjAt+naYZ0B055lcNsbpfEGHVBywDjbxrjDBRz/NYnIms56s6zcbfjGn7sod 3tmy5y07G3RqC82ekvcDob8CSUcyqmxt7dMVxde4= Received: from smtp42.i.mail.ru (smtp42.i.mail.ru [94.100.177.102]) (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 C27F16EC55 for ; Sat, 31 Jul 2021 02:11:30 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org C27F16EC55 Received: by smtp42.i.mail.ru with esmtpa (envelope-from ) id 1m9bf7-0001hW-SV; Sat, 31 Jul 2021 02:11:30 +0300 To: "'Oleg Babin'" , Cc: References: <531544e8-0dd8-8d71-ccfc-4dae2805d498@tarantool.org> In-Reply-To: <531544e8-0dd8-8d71-ccfc-4dae2805d498@tarantool.org> Date: Sat, 31 Jul 2021 02:11:28 +0300 Message-ID: <045e01d78598$3ae26e00$b0a74a00$@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQNJI0ot4YBfKbZyj2tYNprdOv6c+gIKbZKHAzv3X5uoTaeusA== Content-Language: ru X-4EC0790: 10 X-7564579A: 646B95376F6C166E X-77F55803: 4F1203BC0FB41BD941C43E597735A9C366EE405EC28A2001F8302D8429E0DE58182A05F538085040C17502379E1C3B33D264D2B7099626323FE1F699CD31FE9167C8EEFD6F01A445 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7BC08626EA5717D14EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F79006374E88016F1B7D8D248638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D841F30BB0833CA09112F0096820D7B0DE117882F4460429724CE54428C33FAD305F5C1EE8F4F765FC8C7ADC89C2F0B2A5A471835C12D1D9774AD6D5ED66289B52BA9C0B312567BB23117882F446042972877693876707352033AC447995A7AD186FD1C55BDD38FC3FD2E47CDBA5A96583BA9C0B312567BB2376E601842F6C81A19E625A9149C048EE91ADC097FE2C3A081AD9CA79AA6DBBDDD8FC6C240DEA7642DBF02ECDB25306B2B78CF848AE20165D0A6AB1C7CE11FEE3632EDEA9CD5989A3AD7EC71F1DB88427C4224003CC836476EA7A3FFF5B025636E2021AF6380DFAD1A18204E546F3947CB11811A4A51E3B096D1867E19FE1407959CC434672EE6371089D37D7C0E48F6C8AA50765F7900637B8F435DEDE9E76EBEFF80C71ABB335746BA297DBC24807EABDAD6C7F3747799A X-B7AD71C0: AC4F5C86D027EB782CDD5689AFBDA7A213B5FB47DCBC3458F0AFF96BAACF4158235E5A14AD4A4A4625E192CAD1D9E79DB53CE8437368708938807CFCC29EC29F X-C1DE0DAB: 0D63561A33F958A5ADB9C09B623D228DB7E89A2C4E018F0BB6D18682DDAA26B0D59269BC5F550898D99A6476B3ADF6B47008B74DF8BB9EF7333BD3B22AA88B938A852937E12ACA751B940EDA0DFB0535410CA545F18667F91A7EA1CDA0B5A7A0 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D34324A9840C798D5002E319EC1D17461ACAB26B35C7FE74981068A8FB818BA930B1BCFF17038E6472F1D7E09C32AA3244CB050C43B3CA21E70135645896A83EB48F2F5F14F68F1805BFACE5A9C96DEB163 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojWBddABnKmoIn4isK3+OJ+A== X-Mailru-Sender: B5B6A6EBBD94DAD8DF9DB9650A8885C82B2CE89CAEF7605E38381FD5E2B9710D2C555355523812F71EC9E4A2C82A33BC8C24925A86E657CE0C70AEE3C9A96FBAB3D7EE8ED63280BE112434F685709FCF0DA7A0AF5A3A8387 X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH resend v2 10/11] lua, datetime: unixtime, timestamp setters in datetime.lua 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: Timur Safin via Tarantool-patches Reply-To: Timur Safin Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" Hello Oleg, : From: Oleg Babin : Subject: Re: [Tarantool-patches] [PATCH resend v2 10/11] lua, datetime: : unixtime, timestamp setters in datetime.lua : : Thanks for your patch. : : The comment is primarly the same as in previous patch. : : Why do you use such slow approach and creates redundant tables and : function (temporary metatable in fact) : : to use it only once? Please fix it. In previous emails I've shown how I'd deal with loop invariants. Should avoid this redundant temp objects problem. : : : Unixtime and timestamp is great but they loss precision. I think it : should be possible : : go get timestamp with nanoseconds precision since datetime has : nanoseconds precision. I'm not quite get it. Do you want to have nanoseconds exported via newindex? For direct assignments? It's not useful because of available .nsec field. Or may be you meant something else? : : : Also still it's hard how to convert timestamp to datetime value back. Timestamp is something secs.nsec, what looks complicated? I'm confused. Could you please elaborate? : : : On 28.07.2021 13:34, Timur Safin via Tarantool-patches wrote: : > * implemented proper range checks for date attributes values; : > : > * created `.unixtime` attribute, which is alias to `.secs`, : > with corresponding setter/getter; : > : > * similarly to `unixtime`, created virtual `timestamp` attribute : > setter. Which is a convenient way to simultaneously assign : > unixtime (seconds since epoch) and nanoseconds : > : > Part of #5941 Thanks, Timur