From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp17.mail.ru (smtp17.mail.ru [94.100.176.154]) (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 82FCB445320 for ; Wed, 22 Jul 2020 01:42:46 +0300 (MSK) From: Vladislav Shpilevoy Date: Wed, 22 Jul 2020 00:42:42 +0200 Message-Id: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Tarantool-patches] [PATCH 0/2] Make txn_commit() simpler List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tarantool-patches@dev.tarantool.org, gorcunov@gmail.com The patch makes so journal_write() always ends with the transaction completed, with TXN_IS_DONE flag installed. That allows to remove 1 'if' from hot txn_commit() function, and make journal API more straightforward. This change also motivated rework of how limbo commits transactions. Branch: http://github.com/tarantool/tarantool/tree/gerold103/txn_commit-opt Vladislav Shpilevoy (2): txn_limbo: single function to confirm transactions txn: remove TXN_IS_DONE check from txn_commit() src/box/box.cc | 36 +++++++++++++----- src/box/journal.c | 45 +---------------------- src/box/journal.h | 11 ------ src/box/txn.c | 6 +-- src/box/txn_limbo.c | 89 ++++++++++++++++++--------------------------- src/box/wal.c | 2 +- test/unit/suite.ini | 1 + 7 files changed, 68 insertions(+), 122 deletions(-) -- 2.21.1 (Apple Git-122.3)