mirror of
				https://github.com/MariaDB/server.git
				synced 2025-10-31 02:46:29 +01:00 
			
		
		
		
	 31c990ca57
			
		
	
	
	31c990ca57
	
	
	
		
			
			Problem: Statements that write to tables with auto_increment columns
         based on the selection from another table, may lead to master
         and slave going out of sync, as the order in which the rows
         are retrieved from the table may differ on master and slave.
            
Solution: We mark writing to a table with auto_increment table
          based on the rows selected from another table as unsafe. This
          will cause the execution of such statements to throw a warning
          and forces the statement to be logged in ROW if the logging
          format is mixed. 
            
Changes:
       1. All the statements that writes to a table with auto_increment 
          column(s) based on the rows fetched from another table, will now
          be unsafe.
       2. CREATE TABLE with SELECT will now be unsafe.
sql/share/errmsg-utf8.txt:
  Added new warning messages.
sql/sql_base.cc:
  -Created function to check statements that write to 
   tables with auto_increment column and has select.
  -Marked all the statements that write to a table
   with auto_increment column based on rows fetched
   from other table(s) as unsafe.
sql/sql_table.cc:
  mark CREATE TABLE[with auto_increment column] as unsafe.
		
	
			
		
			
				
	
	
		
			66 lines
		
	
	
	
		
			1.4 KiB
		
	
	
	
		
			Text
		
	
	
	
	
	
			
		
		
	
	
			66 lines
		
	
	
	
		
			1.4 KiB
		
	
	
	
		
			Text
		
	
	
	
	
	
| #
 | |
| # Test of auto_increment
 | |
| # BUG#11932
 | |
| #
 | |
| # Bug reported that master and slave get out of sync after TRUNCATE
 | |
| # TABLE.
 | |
| #
 | |
| # Test supplied by Are Casilla
 | |
| 
 | |
| source include/master-slave.inc;
 | |
| call mtr.add_suppression('Unsafe statement written to the binary log using statement format since BINLOG_FORMAT = STATEMENT.');
 | |
| --disable_warnings
 | |
| connection master;
 | |
| drop database if exists test1;
 | |
| --enable_warnings
 | |
| create database test1;
 | |
| use test1;
 | |
| 
 | |
| CREATE TABLE `t1` (
 | |
|   `id` int(10) unsigned NOT NULL auto_increment,
 | |
|   `fname` varchar(100) default NULL,
 | |
|   PRIMARY KEY  (`id`)
 | |
| ) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;
 | |
| 
 | |
| INSERT INTO `t1` VALUES (1, 'blablabla');
 | |
| 
 | |
| CREATE TABLE `t2` (
 | |
|   `id` int(10) NOT NULL auto_increment,
 | |
|   `comment` varchar(255) NOT NULL default '',
 | |
|   PRIMARY KEY  (`id`)
 | |
| ) ENGINE=MyISAM  AUTO_INCREMENT=3 ;
 | |
| 
 | |
| INSERT INTO `t2` VALUES (1, 'testtest 1');
 | |
| INSERT INTO `t2` VALUES (2, 'test 2');
 | |
|   
 | |
| DELIMITER $;
 | |
| CREATE PROCEDURE simpleproc3 ()
 | |
|     NOT DETERMINISTIC
 | |
|     BEGIN
 | |
|     INSERT INTO t1 (fname) (SELECT t2.comment FROM t2 WHERE t2.id = '1');
 | |
|     INSERT INTO t1 (fname) VALUES('test');
 | |
|     END
 | |
|     $
 | |
| DELIMITER ;$
 | |
| 
 | |
| --disable_warnings
 | |
| CALL simpleproc3();
 | |
| --enable_warnings
 | |
| 
 | |
| select * from t2;
 | |
| 
 | |
| TRUNCATE TABLE `t1`;
 | |
| --disable_warnings
 | |
| CALL simpleproc3();
 | |
| --enable_warnings
 | |
| 
 | |
| select * from t1;
 | |
| 
 | |
| sync_slave_with_master;
 | |
| 
 | |
| use test1;
 | |
| select * from t1;
 | |
| 
 | |
| connection master;
 | |
| drop database test1;
 | |
| --source include/rpl_end.inc
 |