Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-15939

Locked_tables_list::reopen_tables is conceptually broken



    • Type: Bug
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 5.5, 10.0, 10.1, 10.2, 10.3
    • Fix Version/s: 10.0, 10.1, 10.2, 10.3
    • Component/s: Locking
    • Labels:


      Locked_tables_list::reopen_tables() is used under LOCK TABLES to reopen all tables that were closed. The failure to reopen might fail the whole statement.

      This is wrong, because reopen might fail for reason completely unrelated to the current statement, because of something that happened earlier. Test case:

      create table t1 (a int);
      create table t2 (b int);
      lock tables t1 write, t2 write, t1 t1_read read;
      -- drop table if exists t1;
      repair table t2 use_frm;
      unlock tables;
      drop table if exists t1;
      drop table if exists t2;

      this works. But if one uncomments drop table, repair will fail — because reopen_tables tries to reopen all closed tables, so it'll try to reopen t1, which no longer exists.

      That's why in some cases the return value of Locked_tables_list::reopen_tables() is ignored (which doesn't help, as it still sets the error status in the thd), and sometimes it is followed by the thd->clear_error() (which is too indiscriminate, and still leaves a warning).

      Ideally, Locked_tables_list::reopen_tables() will take a table as an argument and will reopen just that one table.


          Issue Links



              • Assignee:
                holyfoot Alexey Botchkov
                serg Sergei Golubchik
              • Votes:
                0 Vote for this issue
                2 Start watching this issue


                • Created: