List:Commits« Previous MessageNext Message »
From:Andrei Elkin Date:March 19 2008 5:22pm
Subject:Re: bk commit into 5.0 tree (aelkin:1.2598) BUG#35178
View as plain text  
Sergey, hello.

> Hi Andrei,
>
> as discussed on irc, please add a test case for multiple rows insert for
> both case when insert_id is set and when it is not.
>

I have made it.

> I tend to think that we must use condition that other engines use. That is
> in case of multipl-row insert, call update_auto_increment() only for the
> first row:
>
> if (table->next_number_field && buf == table->record[0])
>

And that's exactly what is needed, is not it?
Indeed, we need the very first value which might be from the explict
set insert_id.

cheers,

Andrei
Thread
bk commit into 5.0 tree (aelkin:1.2598) BUG#35178Andrei Elkin19 Mar
Re: bk commit into 5.0 tree (aelkin:1.2598) BUG#35178Andrei Elkin19 Mar