maria-developers team mailing list archive

[Stewart Smith <stewart@xxxxxxxxxxxxxxxx>]

On Fri, 4 Jun 2010 17:36:49 +0200, Sergei Golubchik <serg@xxxxxxxxxxxx> wrote:
> On Jun 04, Michael Widenius wrote:
> > >>>>> "Sergei" == Sergei  <sergii@xxxxxxxxx> writes:
> > 
> > Sergei> ** Changed in: maria
> > Sergei>    Importance: Undecided => Low
> > 
> > Sergei> -- 
> > Sergei> update is not changing internal auto increment value
> > Sergei> https://bugs.launchpad.net/bugs/314570
> > 
> > Why low ?
> > 
> > Looks like a serious issue that we should get Percona to fix at once!
> 
> Because Heikki said it's not a bug, but intentional InnoDB behavior.
> I'm not sure we should fix it at all. Heikki is certainly not fixing
> it.

So the patch for the Drizzle innobase plugin should almost directly
apply (line offsets will be wrong, but otherwise should be simple).

It all really depends on what the correct behaviour of auto_increment
columns is meant to be.

For comparison, it looks as though NDB has taken the "MyISAM is correct"
path and done the update (I read source, didn't actually try it :)

To me, updating the auto-inc value on UPDATE seems like the correct
thing to do (seeing as we do on update) - hence why i fixed innobase to
behave the same way.
-- 
Stewart Smith