← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/handler.cc

  • Committer: Stewart Smith
  • Date: 2009-05-15 06:57:12 UTC
  • mto: (991.1.5 for-brian)
  • mto: This revision was merged to the branch mainline in revision 1022.
  • Revision ID: stewart@flamingspork.com-20090515065712-bmionylacjmexmmm
https://launchpad.net/bugs/314567
Make sql_mode=NO_AUTO_VALUE_ON_ZERO default for Drizzle.

Also fix DEFAULT keyword handling for auto-increment so that it defaults to
NULL and not 0 so that the following is valid and generates two auto-inc
values:

create table t1 (a int auto_increment primary key)
insert into t1 (a) values (default);
insert into t1 (a) values (default);

Important to note that 0 is no longer magic. So this gives you duplicate
primary key error:

insert into t1 (a) values(0);
insert into t1 (a) values(0);

as you've inserted the explicit value of 0 twice.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/handler.cc
1367
1367
  */
1368
1368
  assert(next_insert_id >= auto_inc_interval_for_cur_row.minimum());
1369
1369
 
1370
 
  if ((nr= table->next_number_field->val_int()) != 0)
 
1370
  /* We check for auto_increment_field_not_null as 0 is an explicit value
 
1371
     for an auto increment column, not a magic value like NULL is.
 
1372
     same as sql_mode=NO_AUTO_VALUE_ON_ZERO */
 
1373
 
 
1374
  if ((nr= table->next_number_field->val_int()) != 0
 
1375
      || table->auto_increment_field_not_null)
1371
1376
  {
1372
1377
    /*
1373
1378
      Update next_insert_id if we had already generated a value in this