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

query start_time not reset when going to sleep.

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 10.0.3, 5.5.31
        • Fix Version/s: 10.0.4, 5.5.32
        • Component/s: None
        • Labels:
          None

          Description

          After executing a query when entering sleep the start time is not reset. So processlist show the time since the last query started not since the start of the sleep.

            Gliffy Diagrams

              Attachments

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                allight Alex Light added a comment -

                a fix

                Show
                allight Alex Light added a comment - a fix
                Hide
                Permalink
                allight Alex Light added a comment -

                updated patch to remove unnessecary second call to set_time

                Show
                allight Alex Light added a comment - updated patch to remove unnessecary second call to set_time
                Hide
                Permalink
                jeremycole Jeremy Cole added a comment -

                This is a pretty good bug and seems to have existed for a very very long time. It does not seem to affect wait_timeout as that is set completely separately as part of vio. This would primarily affect DBAs (visibility) and idle-connection killers that many sites have deployed.

                Show
                jeremycole Jeremy Cole added a comment - This is a pretty good bug and seems to have existed for a very very long time. It does not seem to affect wait_timeout as that is set completely separately as part of vio. This would primarily affect DBAs (visibility) and idle-connection killers that many sites have deployed.
                Hide
                Permalink
                allight Alex Light added a comment -

                final version of the patch. Does this look right to all of you?

                Show
                allight Alex Light added a comment - final version of the patch. Does this look right to all of you?

                  People

                  • Assignee:
                    serg Sergei Golubchik
                    Reporter:
                    allight Alex Light
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    4 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved:

                      Time Tracking

                      Estimated:
                      Original Estimate - Not Specified
                      Not Specified
                      Remaining:
                      Remaining Estimate - 0 minutes
                      0m
                      Logged:
                      Time Spent - 40 minutes
                      40m