Details

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

      Description

      tc_release_table is critical section of table cache. Among other things, it checks if table cache threshold is reached and if so evicts to-be-released table. This is done by comparing tc_count with tc_size.

      tc_count can be atomic and checked beyond of LOCK_open. Besides it is last global variable protected by this lock.

      The cost is as following:

      • tc_add_table() assumed that there is nothing to evict if tc_count > tc_size and skipped expensive LRU lookup. This assumption will be broken and LRU lookup must be done if tc_count >= tc_size.

      A patch for this task hasn't been benchmarked yet.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              svoj Sergey Vojtovich added a comment -

              Sergei, please review patch for this task.

              Show
              svoj Sergey Vojtovich added a comment - Sergei, please review patch for this task.
              Hide
              svoj Sergey Vojtovich added a comment -

              Sergei, please review updated patch.

              Show
              svoj Sergey Vojtovich added a comment - Sergei, please review updated patch.
              Hide
              svoj Sergey Vojtovich added a comment -

              Pushed to 10.0.9, revno: 4003, revision-id: svoj@mariadb.org-20140213061937-dt9f6rx9n13trbv6

              Show
              svoj Sergey Vojtovich added a comment - Pushed to 10.0.9, revno: 4003, revision-id: svoj@mariadb.org-20140213061937-dt9f6rx9n13trbv6

                People

                • Assignee:
                  svoj Sergey Vojtovich
                  Reporter:
                  svoj Sergey Vojtovich
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 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 - 1 hour, 15 minutes
                    1h 15m