[tarantool-patches] Re: [PATCH v8 3/3] sql: remove psql_txn from Vdbe

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Tue Oct 30 22:41:47 MSK 2018


Thanks for the review!

On 30/10/2018 17:30, n.pettik wrote:
> 
>> It makes no sense to store it here, and it has never
>> did.
> 
> Nit: this sentence is grammatically incorrect, I guess.

Sorry, just typo. I removed the second part of the
sentence. Look at the branch.

> 
>> SQL transaction specific things shall be taken
>> from global txn object, as any transaction specific
>> things.
> 
> Patch itself is OK, but I would either separate this patch from patch-set,
> or add "Follow up #xxxx” sign.

Done. I used 'follow up'.




More information about the Tarantool-patches mailing list