From: magnus.blaudd
Date: November 8 2011 10:49am
Subject: bzr push into mysql-5.5-cluster branch (magnus.blaudd:3642 to 3643)
List-Archive: http://lists.mysql.com/commits/141847
Message-Id: <201111081049.pA8AnIdl020314@acsmt357.oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

 3643 magnus.blaudd@stripped	2011-11-08
      ndb schema dist
       - add Ndb_schema_event_handler::is_local_table and use instead of 'ndbcluster_check_if_local_table'

    modified:
      sql/ha_ndbcluster_binlog.cc
 3642 magnus.blaudd@stripped	2011-11-08
      ndb schema dist
       - remove "fall through" cases in 'handle_schema_op', this causes a minro code duplication but
        te code becomes clearer.

    modified:
      sql/ha_ndbcluster_binlog.cc
=== modified file 'sql/ha_ndbcluster_binlog.cc'
--- a/sql/ha_ndbcluster_binlog.cc	2011-11-08 10:16:37 +0000
+++ b/sql/ha_ndbcluster_binlog.cc	2011-11-08 10:34:21 +0000
@@ -2669,6 +2669,12 @@ class Ndb_schema_event_handler {
   }
 
 
+  bool is_local_table(const char* db_name, const char* table_name) const
+  {
+    return ndbcluster_check_if_local_table(db_name, table_name);
+  }
+
+
   void handle_clear_slock(Ndb_schema_op* schema)
   {
     if (!is_post_epoch())
@@ -2788,7 +2794,7 @@ class Ndb_schema_event_handler {
       free_share(&share);
     }
 
-    if (ndbcluster_check_if_local_table(schema->db, schema->name) &&
+    if (is_local_table(schema->db, schema->name) &&
        !Ndb_dist_priv_util::is_distributed_priv_table(schema->db,
                                                       schema->name))
     {
@@ -2817,7 +2823,7 @@ class Ndb_schema_event_handler {
     if (schema->node_id != own_nodeid())
     {
       write_schema_op_to_binlog(m_thd, schema);
-      if (!ndbcluster_check_if_local_table(schema->db, schema->name))
+      if (!is_local_table(schema->db, schema->name))
       {
         mysqld_write_frm_from_ndb(schema->db, schema->name);
       }
@@ -2972,7 +2978,7 @@ class Ndb_schema_event_handler {
         case SOT_RENAME_TABLE_NEW:
         case SOT_DROP_TABLE:
         {
-          if (! ndbcluster_check_if_local_table(schema->db, schema->name))
+          if (!is_local_table(schema->db, schema->name))
           {
             thd_ndb_options.set(TNO_NO_LOCK_SCHEMA_OP);
             const int no_print_error[2]=
@@ -3024,7 +3030,7 @@ class Ndb_schema_event_handler {
             free_share(&share);
 
           thd_ndb_options.set(TNO_NO_LOCK_SCHEMA_OP);
-          if (ndbcluster_check_if_local_table(schema->db, schema->name))
+          if (is_local_table(schema->db, schema->name))
           {
             sql_print_error("NDB Binlog: Skipping locally defined table "
                             "'%s.%s' from binlog schema event '%s' from "
@@ -3042,7 +3048,7 @@ class Ndb_schema_event_handler {
 
         case SOT_CREATE_TABLE:
           thd_ndb_options.set(TNO_NO_LOCK_SCHEMA_OP);
-          if (ndbcluster_check_if_local_table(schema->db, schema->name))
+          if (is_local_table(schema->db, schema->name))
           {
             DBUG_PRINT("info", ("NDB Binlog: Skipping locally defined table '%s.%s'",
                                 schema->db, schema->name));
@@ -3238,7 +3244,7 @@ class Ndb_schema_event_handler {
             share= 0;
           }
 
-          if (ndbcluster_check_if_local_table(schema->db, schema->name))
+          if (is_local_table(schema->db, schema->name))
           {
             DBUG_PRINT("info", ("NDB Binlog: Skipping locally defined table '%s.%s'",
                                 schema->db, schema->name));

No bundle (reason: useless for push emails).