bk commit into 5.1 tree (guilhem:1.2392)

[guilhem]

Below is the list of changes that have just been committed into a local
5.1 repository of guilhem. When guilhem 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.2392 06/05/17 22:16:51 guilhem@stripped +7 -0
  Removing the binlog-show-xid option as it was only a temporary solution until we have
replace-regex which we have now.
  That option was used to suppress the XID from the output of SHOW BINLOG EVENTS (to
create a repeatable testsuite),
  was available only in debug builds, and was explicitely marked as "may be removed in
future versions" in mysqld --help.
  Idea of the removal approved by the replication team.

  sql/mysqld.cc
    1.555 06/05/17 22:16:36 guilhem@stripped +0 -8
    no more need to hide XID in tests output as we have replace-regex

  sql/log_event.h
    1.134 06/05/17 22:16:36 guilhem@stripped +0 -3
    no more need to hide XID in tests output as we have replace-regex

  sql/log_event.cc
    1.221 06/05/17 22:16:36 guilhem@stripped +1 -13
    no more need to hide XID in tests output as we have replace-regex

  mysql-test/mysql-test-run.sh
    1.324 06/05/17 22:16:35 guilhem@stripped +0 -3
    option removed

  mysql-test/mysql-test-run.pl
    1.114 06/05/17 22:16:35 guilhem@stripped +0 -2
    option removed

  mysql-test/extra/binlog_tests/mix_innodb_myisam_binlog.test
    1.28 06/05/17 22:16:35 guilhem@stripped +13 -13
    use replace-regex to remove xid

  mysql-test/extra/binlog_tests/binlog.test
    1.8 06/05/17 22:16:35 guilhem@stripped +3 -3
    use replace-regex to remove xid

# 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:	guilhem
# Host:	gbichot2.local
# Root:	/home/mysql_src/mysql-5.1-new

--- 1.323/mysql-test/mysql-test-run.sh	2006-05-05 19:08:36 +02:00
+++ 1.324/mysql-test/mysql-test-run.sh	2006-05-17 22:16:35 +02:00
@@ -1345,7 +1345,6 @@
           --innodb_data_file_path=ibdata1:128M:autoextend \
 	  --open-files-limit=1024 \
           --log-bin-trust-function-creators \
-          --loose-binlog-show-xid=0 \
 	   $MASTER_40_ARGS \
            $SMALL_SERVER \
            $MASTER_MYSQLD_BINLOG_OPT \
@@ -1369,7 +1368,6 @@
           --language=$LANGUAGE \
           --innodb_data_file_path=ibdata1:128M:autoextend \
           --log-bin-trust-function-creators \
-          --loose-binlog-show-xid=0 \
 	   $MASTER_40_ARGS \
            $SMALL_SERVER \
            $MASTER_MYSQLD_BINLOG_OPT \
@@ -1542,7 +1540,6 @@
           --master-retry-count=10 \
           -O slave_net_timeout=10 \
           --log-bin-trust-function-creators \
-          --loose-binlog-show-xid=0 \
            $SMALL_SERVER \
            $SLAVE_MYSQLD_BINLOG_OPT \
            $EXTRA_SLAVE_MYSQLD_OPT $EXTRA_SLAVE_OPT \

--- 1.220/sql/log_event.cc	2006-04-12 21:19:14 +02:00
+++ 1.221/sql/log_event.cc	2006-05-17 22:16:36 +02:00
@@ -3481,24 +3481,12 @@
   Xid_log_event methods
 **************************************************************************/
 
-#if !defined(DBUG_OFF) && !defined(MYSQL_CLIENT)
-/*
-  This static class member could be removed when mysqltest is made to support
-  a --replace-regex command: then tests which have XIDs in their output can
-  use this command to suppress non-deterministic XID values.
-*/
-my_bool Xid_log_event::show_xid;
-#endif
-
 #if defined(HAVE_REPLICATION) && !defined(MYSQL_CLIENT)
 void Xid_log_event::pack_info(Protocol *protocol)
 {
   char buf[128], *pos;
   pos= strmov(buf, "COMMIT /* xid=");
-#if !defined(DBUG_OFF) && !defined(MYSQL_CLIENT)
-  if (show_xid)
-#endif
-    pos= longlong10_to_str(xid, pos, 10);
+  pos= longlong10_to_str(xid, pos, 10);
   pos= strmov(pos, " */");
   protocol->store(buf, (uint) (pos-buf), &my_charset_bin);
 }

--- 1.133/sql/log_event.h	2006-04-16 03:17:25 +02:00
+++ 1.134/sql/log_event.h	2006-05-17 22:16:36 +02:00
@@ -1221,9 +1221,6 @@
   bool write(IO_CACHE* file);
 #endif
   bool is_valid() const { return 1; }
-#if !defined(DBUG_OFF) && !defined(MYSQL_CLIENT)
-  static my_bool show_xid;
-#endif
 };
 
 /*****************************************************************************

--- 1.554/sql/mysqld.cc	2006-05-12 08:45:17 +02:00
+++ 1.555/sql/mysqld.cc	2006-05-17 22:16:36 +02:00
@@ -4877,14 +4877,6 @@
   {"binlog-ignore-db", OPT_BINLOG_IGNORE_DB,
    "Tells the master that updates to the given database should not be logged tothe binary
log.",
    0, 0, 0, GET_STR, REQUIRED_ARG, 0, 0, 0, 0, 0, 0},
-#if !defined(DBUG_OFF) && !defined(MYSQL_CLIENT)
-  {"binlog-show-xid", OPT_BINLOG_SHOW_XID,
-   "Option used by mysql-test for debugging and testing: "
-   "do not display the XID in SHOW BINLOG EVENTS; "
-   "may be removed in future versions",
-   (gptr*) &Xid_log_event::show_xid, (gptr*) &Xid_log_event::show_xid,
-   0, GET_BOOL, NO_ARG, 1, 0, 0, 0, 0, 0},
-#endif
 #ifdef HAVE_ROW_BASED_REPLICATION
   {"binlog-row-event-max-size", OPT_BINLOG_ROWS_EVENT_MAX_SIZE,
    "The maximum size of a row-based binary log event in bytes. Rows will be "

--- 1.27/mysql-test/extra/binlog_tests/mix_innodb_myisam_binlog.test	2006-02-24 16:28:12
+01:00
+++ 1.28/mysql-test/extra/binlog_tests/mix_innodb_myisam_binlog.test	2006-05-17 22:16:35
+02:00
@@ -30,7 +30,7 @@
 commit;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 delete from t1;
@@ -44,7 +44,7 @@
 rollback;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 delete from t1;
@@ -60,7 +60,7 @@
 commit;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 delete from t1;
@@ -78,7 +78,7 @@
 select a from t1 order by a; # check that savepoints work :)
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 # and when ROLLBACK is not explicit?
@@ -100,7 +100,7 @@
 # logging has been done, we use a user lock.
 select get_lock("a",10);
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 # and when not in a transact1on?
@@ -112,7 +112,7 @@
 insert into t2 select * from t1;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 # Check that when the query updat1ng the MyISAM table is the first in the
@@ -125,13 +125,13 @@
 begin;
 insert into t2 select * from t1;
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 insert into t1 values(11);
 commit;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 
@@ -150,7 +150,7 @@
 commit;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 delete from t1;
@@ -163,7 +163,7 @@
 rollback;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 delete from t1;
@@ -179,7 +179,7 @@
 commit;
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 delete from t1;
@@ -197,7 +197,7 @@
 select a from t1 order by a; # check that savepoints work :)
 
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 
 # Test for BUG#5714, where a MyISAM update in the transaction used to
@@ -258,7 +258,7 @@
 connection con3;
 select get_lock("lock1",60);
 --replace_column 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 do release_lock("lock1");
 drop table t0,t2;

--- 1.113/mysql-test/mysql-test-run.pl	2006-05-12 08:42:00 +02:00
+++ 1.114/mysql-test/mysql-test-run.pl	2006-05-17 22:16:35 +02:00
@@ -2597,7 +2597,6 @@
   mtr_add_arg($args, "%s--character-sets-dir=%s", $prefix, $path_charsetsdir);
   mtr_add_arg($args, "%s--core", $prefix);
   mtr_add_arg($args, "%s--log-bin-trust-function-creators", $prefix);
-  mtr_add_arg($args, "%s--loose-binlog-show-xid=0", $prefix);
   mtr_add_arg($args, "%s--default-character-set=latin1", $prefix);
   mtr_add_arg($args, "%s--language=%s", $prefix, $path_language);
   mtr_add_arg($args, "%s--tmpdir=$opt_tmpdir", $prefix);
@@ -2743,7 +2742,6 @@
   mtr_add_arg($args, "%s--sort_buffer=256K", $prefix);
   mtr_add_arg($args, "%s--max_heap_table_size=1M", $prefix);
   mtr_add_arg($args, "%s--log-bin-trust-function-creators", $prefix);
-  mtr_add_arg($args, "%s--loose-binlog-show-xid=0", $prefix);
 
   if ( $opt_ssl_supported )
   {

--- 1.7/mysql-test/extra/binlog_tests/binlog.test	2006-02-28 09:53:03 +01:00
+++ 1.8/mysql-test/extra/binlog_tests/binlog.test	2006-05-17 22:16:35 +02:00
@@ -22,7 +22,7 @@
 commit;
 # first COMMIT must be Query_log_event, second - Xid_log_event
 --replace_column 2 # 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events from 102;
 drop table t1,t2;
 
@@ -44,8 +44,8 @@
 commit;
 drop table t1;
 --replace_column 2 # 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events in 'master-bin.000001' from 102;
 --replace_column 2 # 5 #
---replace_regex /table_id: [0-9]+/table_id: #/
+--replace_regex /table_id: [0-9]+/table_id: #/ /\/\* xid=.* \*\//\/* xid= *\//
 show binlog events in 'master-bin.000002' from 102;