Details

      Description

      Hello,

      There is a comment about the "capabilities" list in accesslib.php : //TODO: MUC - this could be cached in shared memory, it would eliminate 1 query per page

      While profiling and benchmarking, I noticed that although 1 query could seem small, optimizing it leads to a 2%~3% page load time improvement with apc user cache, because this query returns all rows of a 500 rows table.

      Here is a patch : http://poinsart.com/ftp/ste/moodle-profiling/moodle-speedup2-260dev-20131007.patch

      It adds a "cachedef_cap" entry at the end of the cache definition list in the admin->plugins->caching screen, associating a fast cache to it should give the measurable performance benefit I mentioned above, hopefully without any other visible effects.

      Limits :

      • I did not add the string to describe the cache definition name in the admin screen
      • I'm not really sure whether it's correctly cleared when necessary (i.e. during a moodle upgrade or a plugin install ?), if anyone knows how to test that...

      I preferred to create a separate ticked even if there already is https://tracker.moodle.org/browse/MDL-34242 , I found it quite vague.

      Disclaimer : this is the first time I'm attempting to contribute and I'm still unfamiliar with moodle inner working so don't hesitate to point out any problems in my suggestion

        Attachments

        1. cow_comparison.php
          1 kB
        2. muc_serialisation.png
          muc_serialisation.png
          70 kB
        3. self_cloning.png
          self_cloning.png
          46 kB
        4. test.php
          1 kB

          Issue Links

            Activity

              People

              • Votes:
                2 Vote for this issue
                Watchers:
                10 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  16/Nov/15