Commit 594ba86b authored by unknown's avatar unknown
Browse files

Post-merge fixes.


sql/ha_ndbcluster_binlog.cc:
  Correcting use of bad variable in merge.
sql/sql_base.cc:
  Removing assertion check for mutex in table share.
parent 9a26412c
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -261,7 +261,7 @@ ndbcluster_binlog_open_table(THD *thd, NDB_SHARE *share,
    my_free((gptr) table, MYF(0));
    DBUG_RETURN(error);
  }
  assign_new_table_id(share);
  assign_new_table_id(table_share);
  if (!table->record[1] || table->record[1] == table->record[0])
  {
    table->record[1]= alloc_root(&table->mem_root,
+7 −2
Original line number Diff line number Diff line
@@ -2414,11 +2414,17 @@ void abort_locked_tables(THD *thd,const char *db, const char *table_name)

    share - Pointer to table share structure

  DESCRIPTION

    We are intentionally not checking that share->mutex is locked
    since this function should only be called when opening a table
    share and before it is entered into the table_def_cache (meaning
    that it cannot be fetched by another thread, even accidentally).

  PRE-CONDITION(S)

    share is non-NULL
    The LOCK_open mutex is locked
    The share->mutex is locked

  POST-CONDITION(S)

@@ -2438,7 +2444,6 @@ void assign_new_table_id(TABLE_SHARE *share)
  /* Preconditions */
  DBUG_ASSERT(share != NULL);
  safe_mutex_assert_owner(&LOCK_open);
  safe_mutex_assert_owner(&share->mutex);

  ulong tid= ++last_table_id;                   /* get next id */
  /*