From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf1-f42.google.com (mail-lf1-f42.google.com [209.85.167.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id EDB0446970E for ; Fri, 17 Jan 2020 10:46:26 +0300 (MSK) Received: by mail-lf1-f42.google.com with SMTP id m30so17569425lfp.8 for ; Thu, 16 Jan 2020 23:46:26 -0800 (PST) Date: Fri, 17 Jan 2020 10:46:25 +0300 From: Konstantin Osipov Message-ID: <20200117074625.GE24940@atlas> References: <52419d26967890ace8245f46fdff0604f919a029.1579211601.git.v.shpilevoy@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <52419d26967890ace8245f46fdff0604f919a029.1579211601.git.v.shpilevoy@tarantool.org> Subject: Re: [Tarantool-patches] [PATCH v2 3/3] box: remove dead code from box_process_call/eval() List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vladislav Shpilevoy Cc: tarantool-patches@dev.tarantool.org * Vladislav Shpilevoy [20/01/17 00:57]: > box_process_call/eval() in the end check if there is an > active transaction. If there is, it is rolled back, and > an error is set. > > But rollback is not needed anymore, because anyway in > the end of the request the fiber is stopped, and its > not finished transaction is rolled back. Just setting > of the error is enough. Nice! -- Konstantin Osipov, Moscow, Russia