From: Date: June 14 2007 6:40pm Subject: bk commit into 5.0 tree (tomas:1.2511) BUG#27640 List-Archive: http://lists.mysql.com/commits/28785 X-Bug: 27640 Message-Id: <20070614164037.2050618194033@linux.local> Below is the list of changes that have just been committed into a local 5.0 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@stripped, 2007-06-14 18:40:32+02:00, tomas@stripped +3 -0 Bug #27640 backup id not displayed in the output of "ndb_mgm start backup wait completed" - correction, missing node id mysql-test/r/ndb_backup_print.result@stripped, 2007-06-14 18:40:30+02:00, tomas@stripped +4 -4 Bug #27640 backup id not displayed in the output of "ndb_mgm start backup wait completed" - correction, missing node id mysql-test/t/ndb_backup_print.test@stripped, 2007-06-14 18:40:30+02:00, tomas@stripped +2 -2 Bug #27640 backup id not displayed in the output of "ndb_mgm start backup wait completed" - correction, missing node id ndb/src/mgmclient/CommandInterpreter.cpp@stripped, 2007-06-14 18:40:30+02:00, tomas@stripped +58 -38 Bug #27640 backup id not displayed in the output of "ndb_mgm start backup wait completed" - correction, missing node id # 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: whalegate.ndb.mysql.com # Root: /home/tomas/mysql-5.0-ndb --- 1.1/mysql-test/r/ndb_backup_print.result 2007-06-14 13:40:43 +02:00 +++ 1.2/mysql-test/r/ndb_backup_print.result 2007-06-14 18:40:30 +02:00 @@ -2,8 +2,8 @@ drop table if exists t1,t2,t3,t4,t5,t6,t7,t8,t9,t10; Connected to Management Server at: : Waiting for completed, this may take several minutes -Backup started from node -Backup started from node completed +Node : Backup started from node +Node : Backup started from node completed StartGCP: StopGCP: #Records: #LogRecords: Data: bytes Log: bytes @@ -57,8 +57,8 @@ ); Connected to Management Server at: : Waiting for completed, this may take several minutes -Backup started from node -Backup started from node completed +Node : Backup started from node +Node : Backup started from node completed StartGCP: StopGCP: #Records: #LogRecords: Data: bytes Log: bytes --- 1.1/mysql-test/t/ndb_backup_print.test 2007-06-14 13:40:43 +02:00 +++ 1.2/mysql-test/t/ndb_backup_print.test 2007-06-14 18:40:30 +02:00 @@ -8,7 +8,7 @@ --enable_warnings #NO.1 test output of backup ---exec $NDB_TOOLS_DIR/../src/mgmclient/ndb_mgm -e "start backup" |sed -e 's/[0-9]//g' |sed -e 's/localhost//g' |sed -e 's/\.\.\.*//g' +--exec $NDB_MGM --no-defaults -e "start backup" |sed -e 's/[0-9]//g' |sed -e 's/localhost//g' |sed -e 's/\.\.\.*//g' create table t1 (pk int key @@ -63,4 +63,4 @@ ); #NO.2 test output of backup after some simple SQL operations ---exec $NDB_TOOLS_DIR/../src/mgmclient/ndb_mgm -e "start backup" |sed -e 's/[0-9]//g' |sed -e 's/localhost//g' |sed -e 's/\.\.\.*//g' +--exec $NDB_MGM --no-defaults -e "start backup" |sed -e 's/[0-9]//g' |sed -e 's/localhost//g' |sed -e 's/\.\.\.*//g' --- 1.77/ndb/src/mgmclient/CommandInterpreter.cpp 2007-06-14 14:43:41 +02:00 +++ 1.78/ndb/src/mgmclient/CommandInterpreter.cpp 2007-06-14 18:40:30 +02:00 @@ -772,6 +772,12 @@ /* * print log event from mgmsrv to console screen */ +#define make_uint64(a,b) (((Uint64)(a)) + (((Uint64)(b)) << 32)) +#define Q64(a) make_uint64(event->EVENT.a ## _lo, event->EVENT.a ## _hi) +#define R event->source_nodeid +#define Q(a) event->EVENT.a +#define QVERSION getMajor(Q(version)), getMinor(Q(version)), getBuild(Q(version)) +#define NDB_LE_(a) NDB_LE_ ## a static void printLogEvent(struct ndb_logevent* event) { @@ -779,71 +785,83 @@ /** * NDB_MGM_EVENT_CATEGORY_BACKUP */ - case NDB_LE_BackupStarted: - ndbout_c("Backup %d started from node %d", - event->BackupStarted.backup_id, event->BackupStarted.starting_node); +#undef EVENT +#define EVENT BackupStarted + case NDB_LE_BackupStarted: + ndbout_c("Node %u: Backup %d started from node %d", + R, Q(backup_id), Q(starting_node)); break; +#undef EVENT +#define EVENT BackupFailedToStart case NDB_LE_BackupFailedToStart: - ndbout_c("Backup request from %d failed to start. Error: %d", - event->BackupFailedToStart.starting_node, event->BackupFailedToStart.error); + ndbout_c("Node %u: Backup request from %d failed to start. Error: %d", + R, Q(starting_node), Q(error)); break; +#undef EVENT +#define EVENT BackupCompleted case NDB_LE_BackupCompleted: - ndbout_c("Backup %u started from node %u completed\n" + ndbout_c("Node %u: Backup %u started from node %u completed\n" " StartGCP: %u StopGCP: %u\n" " #Records: %u #LogRecords: %u\n" - " Data: %u bytes Log: %u bytes", - event->BackupCompleted.backup_id, event->BackupCompleted.starting_node, - event->BackupCompleted.start_gci, event->BackupCompleted.stop_gci, - event->BackupCompleted.n_records, event->BackupCompleted.n_log_records, - event->BackupCompleted.n_bytes, event->BackupCompleted.n_log_bytes); + " Data: %u bytes Log: %u bytes", R, + Q(backup_id), Q(starting_node), + Q(start_gci), Q(stop_gci), + Q(n_records), Q(n_log_records), + Q(n_bytes), Q(n_log_bytes)); break; +#undef EVENT +#define EVENT BackupAborted case NDB_LE_BackupAborted: - ndbout_c("Backup %d started from %d has been aborted. Error: %d", - event->BackupAborted.backup_id, event->BackupAborted.starting_node, - event->BackupAborted.error); + ndbout_c("Node %u: Backup %d started from %d has been aborted. Error: %d", + R, Q(backup_id), Q(starting_node), Q(error)); break; /** * NDB_MGM_EVENT_CATEGORY_STARTUP */ +#undef EVENT +#define EVENT NDBStartStarted case NDB_LE_NDBStartStarted: - ndbout_c("Start initiated (version %d.%d.%d)", - getMajor(event->NDBStartStarted.version), - getMinor(event->NDBStartStarted.version), - getBuild(event->NDBStartStarted.version)); + ndbout_c("Node %u: Start initiated (version %d.%d.%d)", + R, QVERSION); break; +#undef EVENT +#define EVENT NDBStartCompleted case NDB_LE_NDBStartCompleted: - ndbout_c("Started (version %d.%d.%d)", - getMajor(event->NDBStartCompleted.version), - getMinor(event->NDBStartCompleted.version), - getBuild(event->NDBStartCompleted.version)); + ndbout_c("Node %u: Started (version %d.%d.%d)", + R, QVERSION); break; +#undef EVENT +#define EVENT NDBStopStarted case NDB_LE_NDBStopStarted: - ndbout_c("%s shutdown initiated", - (event->NDBStopStarted.stoptype == 1 ? "Cluster" : "Node")); + ndbout_c("Node %u: %s shutdown initiated", R, + (Q(stoptype) == 1 ? "Cluster" : "Node")); break; +#undef EVENT +#define EVENT NDBStopCompleted case NDB_LE_NDBStopCompleted: { BaseString action_str(""); BaseString signum_str(""); - getRestartAction(event->NDBStopCompleted.action, action_str); - if (event->NDBStopCompleted.signum) + getRestartAction(Q(action), action_str); + if (Q(signum)) signum_str.appfmt(" Initiated by signal %d.", - event->NDBStopCompleted.signum); - ndbout_c("Node shutdown completed%s.%s", - action_str.c_str(), - signum_str.c_str()); + Q(signum)); + ndbout_c("Node %u: Node shutdown completed%s.%s", + R, action_str.c_str(), signum_str.c_str()); } break; +#undef EVENT +#define EVENT NDBStopForced case NDB_LE_NDBStopForced: { BaseString action_str(""); BaseString reason_str(""); BaseString sphase_str(""); - int signum = event->NDBStopForced.signum; - int error = event->NDBStopForced.error; - int sphase = event->NDBStopForced.sphase; - int extra = event->NDBStopForced.extra; - getRestartAction(event->NDBStopForced.action, action_str); + int signum = Q(signum); + int error = Q(error); + int sphase = Q(sphase); + int extra = Q(extra); + getRestartAction(Q(action), action_str); if (signum) reason_str.appfmt(" Initiated by signal %d.", signum); if (error) @@ -860,13 +878,15 @@ } if (sphase < 255) sphase_str.appfmt(" Occured during startphase %u.", sphase); - ndbout_c("Forced node shutdown completed%s.%s%s", - action_str.c_str(), sphase_str.c_str(), + ndbout_c("Node %u: Forced node shutdown completed%s.%s%s", + R, action_str.c_str(), sphase_str.c_str(), reason_str.c_str()); } break; +#undef EVENT +#define EVENT StopAborted case NDB_LE_NDBStopAborted: - ndbout_c("Node shutdown aborted"); + ndbout_c("Node %u: Node shutdown aborted", R); break; /** * default nothing to print