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

Thread status not changed when applying log events

Details

    • Bug
    • Status: Closed (View Workflow)
    • Minor
    • Resolution: Fixed
    • 10.0.9
    • 10.0.11
    • None
    • None

    Description

      The thread status of the SQL thread is not changed when it starts to apply a log event. The result is that the SQL thread has the status of "System lock" for the whole time it is executing relay log events.

      The status of the thread should be changed when it starts to execute an event.

      Attachments

        Activity

          Ascending order - Click to sort in descending order
          kolbe Kolbe Kegel (Inactive) created issue -
          serg Sergei Golubchik made changes -
          Field Original Value New Value
          Fix Version/s 10.0.11 [ 15200 ]
          kolbe Kolbe Kegel (Inactive) added a comment - 

           +++ sql/log_event.cc    2014-03-19 19:24:26 +0000
          		 
           @@ -9622,6 +9622,7 @@ int Rows_log_event::do_apply_event(rpl_g
          		 
                  goto err;
          		 
                }
          		 
           
          		 
           +    THD_STAGE_INFO(thd, stage_executing);
          		 
                /*

          kolbe Kolbe Kegel (Inactive) added a comment - +++ sql/log_event.cc 2014-03-19 19:24:26 +0000 @@ -9622,6 +9622,7 @@ int Rows_log_event::do_apply_event(rpl_g goto err; }   + THD_STAGE_INFO(thd, stage_executing); /*
          kolbe Kolbe Kegel (Inactive) added a comment - - edited

          That first patch Monty proposed didn't do it, but this other one he proposed does:

          +++ sql/log_event.cc    2014-03-20 16:59:38.112399862 +0000
          		 
          @@ -11183,6 +11183,7 @@ int Rows_log_event::do_apply_event(Relay
          		 
                   break;
          		 
               }
          		 
           
          		 
          +    THD_STAGE_INFO(thd, stage_executing);
          		 
               do {
          		 
           
          		 
                 error= (this->*do_apply_row_ptr)(rli);

          kolbe Kolbe Kegel (Inactive) added a comment - - edited That first patch Monty proposed didn't do it, but this other one he proposed does: +++ sql/log_event.cc 2014-03-20 16:59:38.112399862 +0000 @@ -11183,6 +11183,7 @@ int Rows_log_event::do_apply_event(Relay break; }   + THD_STAGE_INFO(thd, stage_executing); do {   error= (this->*do_apply_row_ptr)(rli);
          monty Michael Widenius made changes -
          Assignee Michael Widenius [ monty ]
          monty Michael Widenius added a comment -

          Fixed.

          monty Michael Widenius added a comment - Fixed.
          monty Michael Widenius made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Closed [ 6 ]
          serg Sergei Golubchik made changes -
          Workflow defaullt [ 36932 ] MariaDB v2 [ 43448 ]
          ratzpo Rasmus Johansson (Inactive) made changes -
          Workflow MariaDB v2 [ 43448 ] MariaDB v3 [ 62553 ]
          serg Sergei Golubchik made changes -
          Workflow MariaDB v3 [ 62553 ] MariaDB v4 [ 147665 ]

          People

            monty Michael Widenius
            kolbe Kolbe Kegel (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Git Integration

                Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.