From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp47.i.mail.ru (smtp47.i.mail.ru [94.100.177.107]) (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 7ECB6445320 for ; Wed, 15 Jul 2020 02:47:32 +0300 (MSK) References: <1594221263-6228-1-git-send-email-alyapunov@tarantool.org> From: Vladislav Shpilevoy Message-ID: Date: Wed, 15 Jul 2020 01:47:30 +0200 MIME-Version: 1.0 In-Reply-To: <1594221263-6228-1-git-send-email-alyapunov@tarantool.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Tarantool-patches] [PATCH v2 00/16] Transaction engine for memtx engine List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aleksandr Lyapunov , tarantool-patches@dev.tarantool.org A lot of new expenses come from struct txn becoming huge. It could be mitigated if you would introduce a new object which would store struct txn, psn, all the new rlists. And which would be created only when needed. However it may be wrong if you will enable the new manager by default with keeping on_yield error. I sent some comments, but I didn't review the patchset whole yet, it is huge.