Below is the list of changes that have just been committed into a local
5.1 repository of root. When root 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, 2007-02-15 13:44:01+08:00, gni@stripped +4 -0
Merge dev3-221.dev.cn.tlan:/home/ngb/mysql/mysql-5.1/mysql-5.1-new-ndb-bj
into dev3-221.dev.cn.tlan:/home/ngb/mysql/mysql-5.1/mysql-5.1-bug24363
MERGE: 1.2336.3.1
storage/ndb/tools/restore/consumer.hpp@stripped, 2007-02-15 13:43:55+08:00, gni@stripped +0 -0
Auto merged
MERGE: 1.12.1.1
storage/ndb/tools/restore/consumer_restore.cpp@stripped, 2007-02-15 13:43:55+08:00, gni@stripped +0 -0
Auto merged
MERGE: 1.38.1.1
storage/ndb/tools/restore/consumer_restore.hpp@stripped, 2007-02-15 13:43:55+08:00, gni@stripped +0 -0
Auto merged
MERGE: 1.15.1.1
storage/ndb/tools/restore/restore_main.cpp@stripped, 2007-02-15 13:43:55+08:00, gni@stripped +0 -0
Auto merged
MERGE: 1.50.1.1
# 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: gni
# Host: dev3-221.dev.cn.tlan
# Root: /home/ngb/mysql/mysql-5.1/mysql-5.1-bug24363/RESYNC
--- 1.51/storage/ndb/tools/restore/restore_main.cpp 2007-02-15 13:44:09 +08:00
+++ 1.52/storage/ndb/tools/restore/restore_main.cpp 2007-02-15 13:44:09 +08:00
@@ -50,6 +50,7 @@
static bool ga_restore_epoch = false;
static bool ga_restore = false;
static bool ga_print = false;
+static bool ga_skip_table_check = false;
static int _print = 0;
static int _print_meta = 0;
static int _print_data = 0;
@@ -91,6 +92,9 @@
NDB_REP_DB "." NDB_APPLY_TABLE " with id 0 will be updated/inserted.",
(gptr*) &ga_restore_epoch, (gptr*) &ga_restore_epoch, 0,
GET_BOOL, NO_ARG, 0, 0, 0, 0, 0, 0 },
+ { "skip-table-check", 's', "Skip table structure check during restore of data",
+ (gptr*) &ga_skip_table_check, (gptr*) &ga_skip_table_check, 0,
+ GET_BOOL, NO_ARG, 0, 0, 0, 0, 0, 0 },
{ "parallelism", 'p',
"No of parallel transactions during restore of data."
"(parallelism can be 1 to 1024)",
@@ -459,6 +463,8 @@
g_options.appfmt(" -n %d", ga_nodeId);
if (_restore_meta)
g_options.appfmt(" -m");
+ if (ga_skip_table_check)
+ g_options.appfmt(" -s");
if (_restore_data)
g_options.appfmt(" -r");
if (ga_restore_epoch)
@@ -589,6 +595,20 @@
{
if(_restore_data || _print_data)
{
+ if (!ga_skip_table_check){
+ for(i=0; i < metaData.getNoOfTables(); i++){
+ if (checkSysTable(metaData, i))
+ {
+ for(Uint32 j= 0; j < g_consumers.size(); j++)
+ if (!g_consumers[j]->table_equal(* metaData[i]))
+ {
+ err << "Restore: Failed to restore data, ";
+ err << metaData[i]->getTableName() << " table structure doesn't match backup ... Exiting " << endl;
+ exitHandler(NDBT_FAILED);
+ }
+ }
+ }
+ }
RestoreDataIterator dataIter(metaData, &free_data_callback);
// Read data file header
--- 1.13/storage/ndb/tools/restore/consumer.hpp 2007-02-15 13:44:09 +08:00
+++ 1.14/storage/ndb/tools/restore/consumer.hpp 2007-02-15 13:44:09 +08:00
@@ -40,6 +40,7 @@
NODE_GROUP_MAP *m_nodegroup_map;
uint m_nodegroup_map_len;
virtual bool has_temp_error() {return false;}
+ virtual bool table_equal(const TableS &) {return true;}
};
#endif
--- 1.39/storage/ndb/tools/restore/consumer_restore.cpp 2007-02-15 13:44:09 +08:00
+++ 1.40/storage/ndb/tools/restore/consumer_restore.cpp 2007-02-15 13:44:09 +08:00
@@ -666,6 +666,62 @@
}
bool
+BackupRestore::table_equal(const TableS &tableS){
+ const char *tablename = tableS.getTableName();
+
+ if(tableS.m_dictTable == NULL){
+ ndbout<<"Table %s has no m_dictTable " << tablename << endl;
+ return false;
+ }
+ /**
+ * Ignore blob tables
+ */
+ if(match_blob(tablename) >= 0)
+ return true;
+
+ const NdbTableImpl & tmptab = NdbTableImpl::getImpl(* tableS.m_dictTable);
+ if ((int) tmptab.m_indexType != (int) NdbDictionary::Index::Undefined){
+ return true;
+ }
+
+ BaseString tmp(tablename);
+ Vector<BaseString> split;
+ if(tmp.split(split, "/") != 3){
+ err << "Invalid table name format " << tablename << endl;
+ return false;
+ }
+
+ m_ndb->setDatabaseName(split[0].c_str());
+ m_ndb->setSchemaName(split[1].c_str());
+
+ NdbDictionary::Dictionary* dict = m_ndb->getDictionary();
+ const NdbDictionary::Table* tab = dict->getTable(split[2].c_str());
+ if(tab == 0){
+ err << "Unable to find table: " << split[2].c_str() << endl;
+ return false;
+ }
+
+ if(tab->getNoOfColumns() != tableS.m_dictTable->getNoOfColumns())
+ {
+ ndbout_c("m_columns.size %d != %d",tab->getNoOfColumns(),
+ tableS.m_dictTable->getNoOfColumns());
+ return false;
+ }
+
+ for(int i = 0; i<tab->getNoOfColumns(); i++)
+ {
+ if(!tab->getColumn(i)->equal(*(tableS.m_dictTable->getColumn(i))))
+ {
+ ndbout_c("m_columns %s != %s",tab->getColumn(i)->getName(),
+ tableS.m_dictTable->getColumn(i)->getName());
+ return false;
+ }
+ }
+
+ return true;
+}
+
+bool
BackupRestore::createSystable(const TableS & tables){
const char *tablename = tables.getTableName();
--- 1.16/storage/ndb/tools/restore/consumer_restore.hpp 2007-02-15 13:44:09 +08:00
+++ 1.17/storage/ndb/tools/restore/consumer_restore.hpp 2007-02-15 13:44:09 +08:00
@@ -73,6 +73,7 @@
virtual bool finalize_table(const TableS &);
virtual bool has_temp_error();
virtual bool createSystable(const TableS & table);
+ virtual bool table_equal(const TableS & table);
virtual bool update_apply_status(const RestoreMetaData &metaData);
void connectToMysql();
bool map_in_frm(char *new_data, const char *data,
| Thread |
|---|
| • bk commit into 5.1 tree (gni:1.2421) | gni | 15 Feb |
