[tarantool-patches] Re: [PATCH v2 1/1] sql: return last_insert_id via IPROTO

[Vladislav Shpilevoy <v.shpilevoy@tarantool.org>]

Hi! Thanks for the fixes!

> commit 67d6dc1cc24a35786fab71222692672c4b01d4be
> Author: Mergen Imeev <imeevma@gmail.com>
> Date:   Tue Jul 31 16:43:38 2018 +0300
> 
>      sql: return last_insert_id via IPROTO
> 
>      After this patch client will get last_insert_id
>      as additional metadata when he executes some
>      statements.
> 
>      Part of #2618
> 
> diff --git a/src/box/sql/vdbeInt.h b/src/box/sql/vdbeInt.h
> index ce97f49..d768eee 100644
> --- a/src/box/sql/vdbeInt.h
> +++ b/src/box/sql/vdbeInt.h
> @@ -400,6 +400,11 @@ struct Vdbe {
>       VdbeFrame *pDelFrame;    /* List of frame objects to free on VM reset */
>       int nFrame;        /* Number of frames in pFrame list */
>       u32 expmask;        /* Binding to these vars invalidates VM */
> +    /*
> +     * True if field last_insert_id of current session was set
> +     * in current statement.
> +     */
> +    bool is_last_insert_id_set;

As I asked, this flag had to be moved to the flags section
of struct Vdbe, next to isPrepareV2, runOnlyOnce etc and occupy
1 bit. I did it for you this time in a separate commit on the
branch. Please, look and squash. And send the whole patchset
under v3 to Nikita P.

>       SubProgram *pProgram;    /* Linked list of all sub-programs used by VM */
>       AuxData *pAuxData;    /* Linked list of auxdata allocations */
>       /* Anonymous savepoint for aborts only */