bzr commit into mysql-5.1 branch (mats:2777) Bug#40116

[Mats Kindahl]

#At file:///home/bzr/bugs/b40116-5.1-5.1.29-rc/

 2777 Mats Kindahl	2008-12-05
      Bug #40116:
      Uncommited changes are replicated and stay on slave after
      rollback on master
      
      Making test slightly more generic and robust.
modified:
  mysql-test/suite/rpl/r/rpl_trigger.result
  mysql-test/suite/rpl/t/rpl_trigger.test

per-file messages:
  mysql-test/suite/rpl/t/rpl_trigger.test
    Adding code to check that InnoDB is available on slave as well.
    Making code slightly more generic.
=== modified file 'mysql-test/suite/rpl/r/rpl_trigger.result'
--- a/mysql-test/suite/rpl/r/rpl_trigger.result	2008-12-03 19:55:49 +0000
+++ b/mysql-test/suite/rpl/r/rpl_trigger.result	2008-12-05 09:23:27 +0000
@@ -988,16 +988,9 @@ after insert on t1
 for each row insert into log values ( new.f );
 set autocommit = 0;
 insert into t1 values ( 1 );
-select * from t1;
-f
-1
-select * from t1;
-f
 rollback;
 Warnings:
 Warning	1196	Some non-transactional changed tables couldn't be rolled back
-select * from t1;
-f
-select * from t1;
-f
+Comparing tables master:test.t1 and slave:test.t1
+Comparing tables master:test.log and slave:test.log
 drop table t1, log;

=== modified file 'mysql-test/suite/rpl/t/rpl_trigger.test'
--- a/mysql-test/suite/rpl/t/rpl_trigger.test	2008-12-03 19:55:49 +0000
+++ b/mysql-test/suite/rpl/t/rpl_trigger.test	2008-12-05 09:23:27 +0000
@@ -480,8 +480,11 @@ sync_slave_with_master;
 # rollback on master
 #
 
+connection master;
 source include/master-slave-reset.inc;
 source include/have_innodb.inc;
+connection slave;
+source include/have_innodb.inc;
 
 connection master;
 create table t1 ( f int ) engine = innodb;
@@ -492,16 +495,15 @@ create trigger tr
 
 set autocommit = 0;
 insert into t1 values ( 1 );
-
-select * from t1;
-sync_slave_with_master;
-select * from t1;
-
-connection master;
 rollback;
-select * from t1;
-sync_slave_with_master;
-select * from t1;
+
+let $diff_table_1=master:test.t1;
+let $diff_table_2=slave:test.t1;
+--source include/diff_tables.inc
+
+let $diff_table_1=master:test.log;
+let $diff_table_2=slave:test.log;
+--source include/diff_tables.inc
 
 connection master;
 drop table t1, log;