• Davi Arnaut's avatar
    Bug#34306: Can't make copy of log tables when server binary log is enabled · 8d6cbf20
    Davi Arnaut authored
    The problem is that when statement-based replication was enabled,
    statements such as INSERT INTO .. SELECT FROM .. and CREATE TABLE
    .. SELECT FROM need to grab a read lock on the source table that
    does not permit concurrent inserts, which would in turn be denied
    if the source table is a log table because log tables can't be
    locked exclusively.
    
    The solution is to not take such a lock when the source table is
    a log table as it is unsafe to replicate log tables under statement
    based replication. Furthermore, the read lock that does not permits
    concurrent inserts is now only taken if statement-based replication
    is enabled and if the source table is not a log table.
    
    include/thr_lock.h:
      Introduce yet another lock type that my get upgraded depending
      on the binary log format. This is not a optimal solution but
      can be easily improved later.
    mysql-test/r/log_tables.result:
      Add test case result for Bug#34306
    mysql-test/suite/binlog/r/binlog_stm_row.result:
      Add test case result for Bug#34306
    mysql-test/suite/binlog/t/binlog_stm_row.test:
      Add test case for Bug#34306
    mysql-test/t/log_tables.test:
      Add test case for Bug#34306
    sql/lock.cc:
      Assert that TL_READ_DEFAULT is not a real lock type.
    sql/mysql_priv.h:
      Export new function.
    sql/mysqld.cc:
      Remove using_update_log.
    sql/sql_base.cc:
      Introduce function that returns the appropriate read lock type
      depending on how the statement is going to be replicated. It will
      only take a TL_READ_NO_INSERT log if the binary is enabled and the
      binary log format is statement-based and the table is not a log table.
    sql/sql_parse.cc:
      Remove using_update_log.
    sql/sql_update.cc:
      Use new function to choose read lock type.
    sql/sql_yacc.yy:
      The lock type is now decided at open_tables time. This old behavior was
      actually misleading as the binary log format can be dynamically switched
      and this would not change for statements that have already been parsed
      when the binary log format is changed (ie: prepared statements).
    8d6cbf20
lock.cc 48.2 KB