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

Account for queries handled by query-cache in USER_STATISTICS (and in HOST_STATISTICS)

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      [Affected version: at least 5.5.28a-MariaDB-log]

      When a user performs queries handled by query-cache theses are not taken into account by USER_STATISTICS (and probably neither by HOST_STATISTICS).

      They should properly increment COMMAND_SELECT, SENT_ROWS and when appropriate EMPTY_QUERIES.

      Without updating those a user that performs the same SELECT over and over, each time hitting query cache will appear as performing no queries, just burning CPU time and exchanging bytes.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              sanja Oleksandr Byelkin added a comment -

              QC should be transparent, so SELECT_COMMANDS should count QC answers.

              Show
              sanja Oleksandr Byelkin added a comment - QC should be transparent, so SELECT_COMMANDS should count QC answers.
              Hide
              sanja Oleksandr Byelkin added a comment -

              Normal execution is fixed. now have to check SP and PS.

              Show
              sanja Oleksandr Byelkin added a comment - Normal execution is fixed. now have to check SP and PS.
              Hide
              sanja Oleksandr Byelkin added a comment -

              EMPTY_QUERIES (both) counts queries with empty results (So I was wrong initially).

              Show
              sanja Oleksandr Byelkin added a comment - EMPTY_QUERIES (both) counts queries with empty results (So I was wrong initially).
              Hide
              sanja Oleksandr Byelkin added a comment -

              Committed for review.

              Show
              sanja Oleksandr Byelkin added a comment - Committed for review.
              Hide
              sanja Oleksandr Byelkin added a comment -

              pushed to 5.5

              Show
              sanja Oleksandr Byelkin added a comment - pushed to 5.5

                People

                • Assignee:
                  sanja Oleksandr Byelkin
                  Reporter:
                  bprest Bruno Prémont
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: