mariadb/sql-common
Brandon Nesterenko a83c7ab1ea MDEV-11853: semisync thread can be killed after sync binlog but before ACK in the sync state
Problem:
========
If a primary is shutdown during an active semi-sync connection
during the period when the primary is awaiting an ACK, the primary
hard kills the active communication thread and does not ensure the
transaction was received by a replica. This can lead to an
inconsistent replication state.

Solution:
========
During shutdown, the primary should wait for an ACK or timeout
before hard killing a thread which is awaiting a communication. We
extend the `SHUTDOWN WAIT FOR SLAVES` logic to identify and ignore
any threads waiting for a semi-sync ACK in phase 1. Then, before
stopping the ack receiver thread, the shutdown is delayed until all
waiting semi-sync connections receive an ACK or time out. The
connections are then killed in phase 2.

Notes:
 1) There remains an unresolved corner case that affects this
patch. MDEV-28141: Slave crashes with Packets out of order when
connecting to a shutting down master. Specifically, If a slave is
connecting to a master which is actively shutting down, the slave
can crash with a "Packets out of order" assertion error. To get
around this issue in the MTR tests, the primary will wait a small
amount of time before phase 1 killing threads to let the replicas
safely stop (if applicable).
 2) This patch also fixes MDEV-28114: Semi-sync Master ACK Receiver
Thread Can Error on COM_QUIT

Reviewed By
============
Andrei Elkin <andrei.elkin@mariadb.com>
2022-04-22 12:59:54 -06:00
..
client.c MDEV-11853: semisync thread can be killed after sync binlog but before ACK in the sync state 2022-04-22 12:59:54 -06:00
client_plugin.c Merge 10.2 into 10.3 2019-10-09 13:25:11 +03:00
conf_to_src.c Merge 10.1 into 10.2 2019-05-13 17:54:04 +03:00
errmsg.c Merge 10.1 into 10.2 2019-05-13 17:54:04 +03:00
my_time.c Merge 10.3 into 10.4, except for MDEV-20265 2019-08-23 08:06:17 +03:00
my_user.c Merge 10.2 into 10.3 2019-05-14 17:18:46 +03:00
mysql_async.c MDEV-15091 : Windows, 64bit: reenable and fix warning C4267 (conversion from 'size_t' to 'type', possible loss of data) 2018-02-06 12:55:58 +00:00
pack.c Merge 10.1 into 10.2 2019-05-13 17:54:04 +03:00