bk commit into 5.1 tree (tomas:1.2083) BUG#24503

[tomas]

Below is the list of changes that have just been committed into a local
5.1 repository of tomas. When tomas 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
  1.2083 07/01/04 12:44:37 tomas@poseidon. +6 -0
    ndb - bug#24503
      Fix scan filter on bit types
      (note should probably be enabled in condition pushdown aswell)

  storage/ndb/src/ndbapi/ndberror.c
    1.50 07/01/04 12:44:28 tomas@poseidon. +1 -1
        Add new error code
        Remove unused error code

  storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp
    1.22 07/01/04 12:44:28 tomas@poseidon. +10 -0
        Add correct handling of unsupported type

  storage/ndb/src/kernel/blocks/dbtup/DbtupAbort.cpp
    1.4 07/01/04 12:44:28 tomas@poseidon. +5 -0
        Add new error code

  storage/ndb/src/kernel/blocks/dbtup/Dbtup.hpp
    1.33 07/01/04 12:44:28 tomas@poseidon. +1 -0
        Add new error code

  storage/ndb/src/kernel/blocks/dbtc/Dbtc.hpp
    1.37 07/01/04 12:44:28 tomas@poseidon. +0 -1
    remove unused error code

  storage/ndb/src/common/util/NdbSqlUtil.cpp
    1.31 07/01/04 12:44:27 tomas@poseidon. +12 -7
        Add support for comparing bit types
        remove debug printouts

# 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:	tomas
# Host:	poseidon.
# Root:	/home/tomas/mysql-5.1-wl2325-5.0

--- 1.30/storage/ndb/src/common/util/NdbSqlUtil.cpp	2005-04-08 02:43:59 +02:00
+++ 1.31/storage/ndb/src/common/util/NdbSqlUtil.cpp	2007-01-04 12:44:27 +01:00
@@ -136,7 +136,7 @@
   },
   { // 22
     Type::Bit,
-    NULL,
+    cmpBit,
     NULL
   },
   { // 23
@@ -679,6 +679,17 @@
 }
 
 int
+NdbSqlUtil::cmpBit(const void* info, const void* p1, unsigned n1, const void* p2,
unsigned n2, bool full)
+{ 
+  Uint32 n = (n1 < n2) ? n1 : n2;
+  char* c1 = (char*)p1;
+  char* c2 = (char*)p2;
+  int ret = memcmp(p1, p2, n);
+  return ret;
+}
+
+
+int
 NdbSqlUtil::cmpTime(const void* info, const void* p1, unsigned n1, const void* p2,
unsigned n2, bool full)
 {
   if (n2 >= 3) {
@@ -698,12 +709,6 @@
 }
 
 // not yet
-int
-NdbSqlUtil::cmpBit(const void* info, const void* p1, unsigned n1, const void* p2,
unsigned n2, bool full)
-{
-  assert(false);
-  return 0;
-}
 
 int
 NdbSqlUtil::cmpLongvarchar(const void* info, const void* p1, unsigned n1, const void* p2,
unsigned n2, bool full)

--- 1.36/storage/ndb/src/kernel/blocks/dbtc/Dbtc.hpp	2006-10-10 16:13:27 +02:00
+++ 1.37/storage/ndb/src/kernel/blocks/dbtc/Dbtc.hpp	2007-01-04 12:44:28 +01:00
@@ -139,7 +139,6 @@
 
 #define ZNOT_FOUND 626
 #define ZALREADYEXIST 630
-#define ZINCONSISTENTHASHINDEX 892
 #define ZNOTUNIQUE 893
 
 #define ZINVALID_KEY 290

--- 1.32/storage/ndb/src/kernel/blocks/dbtup/Dbtup.hpp	2006-06-28 16:40:20 +02:00
+++ 1.33/storage/ndb/src/kernel/blocks/dbtup/Dbtup.hpp	2007-01-04 12:44:28 +01:00
@@ -196,6 +196,7 @@
 #define ZTRY_TO_UPDATE_ERROR 888
 #define ZCALL_ERROR 890
 #define ZTEMPORARY_RESOURCE_FAILURE 891
+#define ZUNSUPPORTED_BRANCH 892
 
 #define ZSTORED_SEIZE_ATTRINBUFREC_ERROR 873 // Part of Scan
 

--- 1.3/storage/ndb/src/kernel/blocks/dbtup/DbtupAbort.cpp	2005-04-08 02:44:04 +02:00
+++ 1.4/storage/ndb/src/kernel/blocks/dbtup/DbtupAbort.cpp	2007-01-04 12:44:28 +01:00
@@ -444,6 +444,11 @@
     }//if
     break;
 
+  case 40:
+    ljam();
+    terrorCode = ZUNSUPPORTED_BRANCH;
+    break;
+    
   default:
     ndbrequire(false);
     break;

--- 1.21/storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp	2005-09-20 12:23:32 +02:00
+++ 1.22/storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp	2007-01-04 12:44:28 +01:00
@@ -1869,6 +1869,11 @@
             // NULL==NULL and NULL<not-NULL
             res1 = r1_null && r2_null ? 0 : r1_null ? -1 : 1;
           } else {
+	    jam();
+	    if (unlikely(sqlType.m_cmp == 0))
+	    {
+	      return TUPKEY_abort(signal, 40);
+	    }
             res1 = (*sqlType.m_cmp)(cs, s1, attrLen, s2, argLen, true);
           }
 	} else {
@@ -1876,6 +1881,11 @@
             // NULL like NULL is true (has no practical use)
             res1 =  r1_null && r2_null ? 0 : -1;
           } else {
+	    jam();
+	    if (unlikely(sqlType.m_like == 0))
+	    {
+	      return TUPKEY_abort(signal, 40);
+	    }
             res1 = (*sqlType.m_like)(cs, s1, attrLen, s2, argLen);
           }
         }

--- 1.49/storage/ndb/src/ndbapi/ndberror.c	2006-03-27 18:55:41 +02:00
+++ 1.50/storage/ndb/src/ndbapi/ndberror.c	2007-01-04 12:44:28 +01:00
@@ -218,7 +218,6 @@
   /**
    * Internal errors
    */
-  { 892,  DMEC, IE, "Inconsistent hash index. The index needs to be dropped and
recreated" },
   { 896,  DMEC, IE, "Tuple corrupted - wrong checksum or column data in invalid format"
},
   { 901,  DMEC, IE, "Inconsistent ordered index. The index needs to be dropped and
recreated" },
   { 202,  DMEC, IE, "202" },
@@ -285,6 +284,7 @@
   { 885,  DMEC, AE, "Stack underflow in interpreter" },
   { 886,  DMEC, AE, "More than 65535 instructions executed in interpreter" },
   { 897,  DMEC, AE, "Update attempt of primary key via ndbcluster internal api (if this
occurs via the MySQL server it is a bug, please report)" },
+  { 892,  DMEC, AE, "Unsupported type in scan filter" },
   { 4256, DMEC, AE, "Must call Ndb::init() before this function" },
   { 4257, DMEC, AE, "Tried to read too much - too many getValue calls" },