bk commit into 5.1 tree (mskold:1.2314) BUG#21072

[Martin Skold]

Below is the list of changes that have just been committed into a local
5.1 repository of marty. When marty does a push these changes will
be propagated to the main repository and, within 24 hours after the
push, to the public repository.
For information on how to access the public repository
see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html

ChangeSet@stripped, 2006-10-16 12:14:38+02:00, mskold@stripped +4 -0
   Bug #21072  Duplicate key error in NDB references wrong key: Added new printout for
duplicate key error when key is unknown

  mysql-test/r/ndb_charset.result@stripped, 2006-10-16 12:12:45+02:00, mskold@stripped +4 -4
     Bug #21072  Duplicate key error in NDB references wrong key: Added new printout for
duplicate key error when key is unknown

  mysql-test/r/ndb_index_unique.result@stripped, 2006-10-16 12:12:45+02:00, mskold@stripped
+6 -6
     Bug #21072  Duplicate key error in NDB references wrong key: Added new printout for
duplicate key error when key is unknown

  mysql-test/r/ndb_update.result@stripped, 2006-10-16 12:14:11+02:00, mskold@stripped +1 -1
     Bug #21072  Duplicate key error in NDB references wrong key: Added new printout for
duplicate key error when key is unknown

  sql/handler.cc@stripped, 2006-10-16 12:12:44+02:00, mskold@stripped +20 -8
     Bug #21072  Duplicate key error in NDB references wrong key: Added new printout for
duplicate key error when key is unknown

# This is a BitKeeper patch.  What follows are the unified diffs for the
# set of deltas contained in the patch.  The rest of the patch, the part
# that BitKeeper cares about, is below these diffs.
# User:	mskold
# Host:	linux.site
# Root:	/windows/Linux_space/MySQL/mysql-5.1

--- 1.274/sql/handler.cc	2006-10-16 12:14:49 +02:00
+++ 1.275/sql/handler.cc	2006-10-16 12:14:49 +02:00
@@ -2031,16 +2031,28 @@ void handler::print_keydup_error(uint ke
   /* Write the duplicated key in the error message */
   char key[MAX_KEY_LENGTH];
   String str(key,sizeof(key),system_charset_info);
-  /* Table is opened and defined at this point */
-  key_unpack(&str,table,(uint) key_nr);
-  uint max_length=MYSQL_ERRMSG_SIZE-(uint) strlen(msg);
-  if (str.length() >= max_length)
+
+  if (key_nr == MAX_KEY)
   {
-    str.length(max_length-4);
-    str.append(STRING_WITH_LEN("..."));
+    /* Key is unknown */
+    str.length(0);
+    key_nr= -1;
+    my_printf_error(ER_DUP_ENTRY, msg,
+		    MYF(0), str.c_ptr(), "*UNKNOWN*");
+  }
+  else
+  {
+    /* Table is opened and defined at this point */
+    key_unpack(&str,table,(uint) key_nr);
+    uint max_length=MYSQL_ERRMSG_SIZE-(uint) strlen(msg);
+    if (str.length() >= max_length)
+    {
+      str.length(max_length-4);
+      str.append(STRING_WITH_LEN("..."));
+    }
+    my_printf_error(ER_DUP_ENTRY, msg,
+		    MYF(0), str.c_ptr(), table->key_info[key_nr].name);
   }
-  my_printf_error(ER_DUP_ENTRY, msg,
-                  MYF(0), str.c_ptr(), table->key_info[key_nr].name);
 }
 
 

--- 1.27/mysql-test/r/ndb_index_unique.result	2006-10-16 12:14:49 +02:00
+++ 1.28/mysql-test/r/ndb_index_unique.result	2006-10-16 12:14:49 +02:00
@@ -22,7 +22,7 @@ select * from t1 where b = 4 order by a;
 a	b	c
 3	4	6
 insert into t1 values(8, 2, 3);
-ERROR 23000: Duplicate entry '8' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select * from t1 order by a;
 a	b	c
 1	2	3
@@ -93,7 +93,7 @@ a	b	c
 1	1	1
 4	4	NULL
 insert into t1 values(5,1,1);
-ERROR 23000: Duplicate entry '5' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 drop table t1;
 CREATE TABLE t2 (
 a int unsigned NOT NULL PRIMARY KEY,
@@ -116,7 +116,7 @@ select * from t2 where b = 4 order by a;
 a	b	c
 3	4	6
 insert into t2 values(8, 2, 3);
-ERROR 23000: Duplicate entry '8' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select * from t2 order by a;
 a	b	c
 1	2	3
@@ -181,7 +181,7 @@ pk	a
 3	NULL
 4	4
 insert into t1 values (5,0);
-ERROR 23000: Duplicate entry '5' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select * from t1 order by pk;
 pk	a
 -1	NULL
@@ -214,7 +214,7 @@ pk	a	b	c
 0	NULL	18	NULL
 1	3	19	abc
 insert into t2 values(2,3,19,'abc');
-ERROR 23000: Duplicate entry '2' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select * from t2 order by pk;
 pk	a	b	c
 -1	1	17	NULL
@@ -634,7 +634,7 @@ create table t1 (a int primary key, b va
 engine=ndb charset=utf8;
 insert into t1 values (1, repeat(_utf8 0xe288ab6474, 200));
 insert into t1 values (2, repeat(_utf8 0xe288ab6474, 200));
-ERROR 23000: Duplicate entry '2' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select a, sha1(b) from t1;
 a	sha1(b)
 1	08f5d02c8b8bc244f275bdfc22c42c5cab0d9d7d

--- 1.12/mysql-test/r/ndb_update.result	2006-10-16 12:14:49 +02:00
+++ 1.13/mysql-test/r/ndb_update.result	2006-10-16 12:14:49 +02:00
@@ -18,7 +18,7 @@ pk1	b	c
 2	2	2
 4	1	1
 UPDATE t1 set pk1 = 1, c = 2 where pk1 = 4;
-ERROR 23000: Duplicate entry '1' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select * from t1 order by pk1;
 pk1	b	c
 0	0	0

--- 1.12/mysql-test/r/ndb_charset.result	2006-10-16 12:14:49 +02:00
+++ 1.13/mysql-test/r/ndb_charset.result	2006-10-16 12:14:49 +02:00
@@ -112,9 +112,9 @@ unique key(a)
 ) engine=ndb;
 insert into t1 values(1, 'aAa');
 insert into t1 values(2, 'aaa');
-ERROR 23000: Duplicate entry '2' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 insert into t1 values(3, 'AAA');
-ERROR 23000: Duplicate entry '3' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select * from t1 order by p;
 p	a
 1	aAa
@@ -138,9 +138,9 @@ unique key(a)
 ) engine=ndb;
 insert into t1 values (1,'A'),(2,'b '),(3,'C  '),(4,'d      '),(5,'E'),(6,'f');
 insert into t1 values(99,'b');
-ERROR 23000: Duplicate entry '99' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 insert into t1 values(99,'a         ');
-ERROR 23000: Duplicate entry '99' for key 'PRIMARY'
+ERROR 23000: Duplicate entry '' for key '*UNKNOWN*'
 select a,length(a) from t1 order by a;
 a	length(a)
 A	1