From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <tarantool-patches-bounce@freelists.org>
Received: from localhost (localhost [127.0.0.1])
	by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id C899323FD2
	for <tarantool-patches@freelists.org>; Mon, 21 May 2018 17:46:34 -0400 (EDT)
Received: from turing.freelists.org ([127.0.0.1])
	by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id sryy1-f1mebS for <tarantool-patches@freelists.org>;
	Mon, 21 May 2018 17:46:34 -0400 (EDT)
Received: from smtp40.i.mail.ru (smtp40.i.mail.ru [94.100.177.100])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id 822C723FBD
	for <tarantool-patches@freelists.org>; Mon, 21 May 2018 17:46:34 -0400 (EDT)
Subject: [tarantool-patches] Re: [PATCH 1/1] sql: fix out of time auto commit
 mode detection
References: <0fecccaea1baa046854e58e9ebffeb207103e7c3.1526938154.git.v.shpilevoy@tarantool.org>
 <1526939115.721063905@f481.i.mail.ru>
From: Vladislav Shpilevoy <v.shpilevoy@tarantool.org>
Message-ID: <72eccb80-83ec-bcda-56b0-8abe69f0cccb@tarantool.org>
Date: Tue, 22 May 2018 00:46:32 +0300
MIME-Version: 1.0
In-Reply-To: <1526939115.721063905@f481.i.mail.ru>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: tarantool-patches-bounce@freelists.org
Errors-to: tarantool-patches-bounce@freelists.org
Reply-To: tarantool-patches@freelists.org
List-help: <mailto:ecartis@freelists.org?Subject=help>
List-unsubscribe: <tarantool-patches-request@freelists.org?Subject=unsubscribe>
List-software: Ecartis version 1.0.0
List-Id: tarantool-patches <tarantool-patches.freelists.org>
List-subscribe: <tarantool-patches-request@freelists.org?Subject=subscribe>
List-owner: <mailto:>
List-post: <mailto:tarantool-patches@freelists.org>
List-archive: <http://www.freelists.org/archives/tarantool-patches>
To: Nikita Pettik <korablev@tarantool.org>, tarantool-patches@freelists.org

Hello. Thanks for review!

On 22/05/2018 00:45, Nikita Pettik wrote:
>     + * Once per execution time prepare the programm: detect
> 
> Nitpicking: 'program'.

@@ -5352,7 +5352,7 @@ case OP_Init: {          /* jump */
         assert(pOp->p4.z==0 || strncmp(pOp->p4.z, "-" "- ", 3)==0);
         assert(pOp==p->aOp);  /* Always instruction 0 */
         /*
-        * Once per execution time prepare the programm: detect
+        * Once per execution time prepare the program: detect
          * autocommit, create SQL specific transaction things. To


> 
> LGTM.
> 
> -- 
> Nikita Pettik
>