• Kristian Nielsen's avatar
    MDEV-7668: Intermediate master groups CREATE TEMPORARY with INSERT, causing... · 96784eb1
    Kristian Nielsen authored
    MDEV-7668: Intermediate master groups CREATE TEMPORARY with INSERT, causing parallel replication failure
    
    Parallel replication depends on locking (table locks, row locks, etc.) to
    prevent two conflicting transactions from running and committing in parallel.
    But temporary tables are designed to be visible only to one thread, and have
    no such locking.
    
    In the concrete issue, an intermediate master could commit a CREATE TEMPORARY
    TABLE in the same group commit as in INSERT into that table. Thus, a
    lower-level master could attempt to run them in parallel and get an error.
    
    More generally, we need protection from parallel replication trying to run
    transactions in parallel that access a common temporary table.
    
    This patch simply causes use of a temporary table from parallel replication
    to wait for all previous transactions to commit, serialising the replication
    at that point.
    
    (A more fine-grained locking could be added later, possibly. However,
    using temporary tables in statement-based replication is in any case
    normally undesirable; for example a restart of the server will lose
    temporary tables and can break replication).
    
    Note that row-based replication is not affected, as it does not do any
    temporary tables on the slave-side.
    
    This patch also cleans up the locking around protecting the list of
    temporary tables in Relay_log_info. This used to take the
    rli->data_lock at the end of every statement, which is very bad for
    concurrency. With this patch, the lock is not taken unless temporary
    tables (with statement-based binlogging) are in use on the slave.
    96784eb1
sql_base.cc 312 KB