mirror of
https://github.com/MariaDB/server.git
synced 2025-07-20 18:28:14 +02:00
72e1cc8f52
In Log_event::read_log_event(), don't use IO_CACHE::error of the relay log's
IO_CACHE to signal an error back to the caller. When reading the active
relay log, this flag is also being used by the IO thread, and setting it can
randomly cause the IO thread to wrongly detect IO error on writing and
permanently disable the relay log.
This was seen sporadically in test case rpl.rpl_from_mysql80. The read
error set by the SQL thread in the IO_CACHE would be interpreted as a
write error by the IO thread, which would cause it to throw a fatal
error and close the relay log. And this would later cause CHANGE
MASTER to try to purge a closed relay log, resulting in nullptr crash.
SQL thread is not able to parse an event read from the relay log. This
can happen like here when replicating unknown events from a MySQL master,
potentially also for other reasons.
Also fix a mistake in my_b_flush_io_cache() introduced back in 2001
(fa09f2cd7e) where my_b_flush_io_cache() could wrongly return an error set
in IO_CACHE::error, even if the flush operation itself succeeded.
Also fix another sporadic failure in rpl.rpl_from_mysql80 where the outout
of MASTER_POS_WAIT() depended on timing of SQL and IO thread.
Reviewed-by: Monty <monty@mariadb.org>
Reviewed-by: Andrei Elkin <andrei.elkin@mariadb.com>
Signed-off-by: Kristian Nielsen <knielsen@knielsen-hq.org>
50 lines
2.3 KiB
Text
50 lines
2.3 KiB
Text
include/master-slave.inc
|
|
[connection master]
|
|
connection slave;
|
|
include/stop_slave.inc
|
|
connection master;
|
|
include/rpl_stop_server.inc [server_number=1]
|
|
include/rpl_start_server.inc [server_number=1]
|
|
connection slave;
|
|
CHANGE MASTER TO Master_log_file='master-bin.000001', Master_log_pos=4, Master_use_gtid=No;
|
|
START SLAVE IO_THREAD;
|
|
include/wait_for_slave_io_to_start.inc
|
|
START SLAVE UNTIL Master_log_file='master-bin.000001', Master_log_pos= 1178;
|
|
SELECT MASTER_POS_WAIT('master-bin.000001', 1178, 60);
|
|
SELECT * FROM t1 ORDER BY a;
|
|
a b c
|
|
1 0
|
|
2 0 hulu
|
|
3 0 bulu
|
|
include/wait_for_slave_sql_to_stop.inc
|
|
START SLAVE;
|
|
include/wait_for_slave_sql_error.inc [errno=1594]
|
|
STOP SLAVE IO_THREAD;
|
|
include/wait_for_slave_io_to_stop.inc
|
|
CHANGE MASTER TO Master_log_file='master-bin.000001', Master_log_pos=2297;
|
|
START SLAVE IO_THREAD;
|
|
START SLAVE SQL_THREAD;
|
|
include/wait_for_slave_io_to_start.inc
|
|
include/wait_for_slave_sql_error.inc [errno=1594]
|
|
SELECT * FROM t1 ORDER BY a;
|
|
a b c
|
|
1 0
|
|
2 0 hulu
|
|
3 0 bulu
|
|
4 0 skip
|
|
5 0 after compressed
|
|
SELECT * FROM t2 ORDER BY a;
|
|
a b
|
|
1 {"a": "hulu", "b": "[zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy][zyzzy]", "c": "bulu"}
|
|
STOP SLAVE IO_THREAD;
|
|
include/wait_for_slave_io_to_stop.inc
|
|
CHANGE MASTER TO Master_log_file='master-bin.000002', Master_log_pos=4;
|
|
START SLAVE IO_THREAD;
|
|
START SLAVE SQL_THREAD;
|
|
include/wait_for_slave_io_to_start.inc
|
|
include/wait_for_slave_sql_to_start.inc
|
|
DROP TABLE t1, t2;
|
|
CALL mtr.add_suppression('TRANSACTION_PAYLOAD_EVENT event. You can avoid this event by specifying');
|
|
CALL mtr.add_suppression('PARTIAL_UPDATE_ROWS_EVENT event. You can avoid this event by specifying');
|
|
connection master;
|
|
include/rpl_end.inc
|