Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-40581

backupidsexists and backupidscache could be used more effectively

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Pass all PHPUnit tests on ALL databases.

      Copy the file backup_restore_testcase.php into backup/tests/ folder, and run the phpunit test  to compare performance between master and new branch.

      Run command:

      vendor/bin/phpunit backup/tests/backup_restore_testcase.php --no-configuration --dont-report-useless-tests --bootstrap lib/phpunit/bootstrap.php
      

      Show
      Pass all PHPUnit tests on ALL databases. Copy the file backup_restore_testcase.php into backup/tests/ folder, and run the phpunit test  to compare performance between master and new branch. Run command: vendor/bin/phpunit backup/tests/backup_restore_testcase.php --no-configuration --dont-report-useless-tests --bootstrap lib/phpunit/bootstrap.php
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_310_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
    • Pull Master Branch:
      MDL-40581_master_v6

      Description

      The restore process uses backupidsexists and backupidscache to store the existence and data from the database to reduce overhead when importing. The cache was installed as part of MDL-27120. It is slanted to be replaced by MUC.

      This issue will explore the performance benefit/cost of MUC and also improvements to the way these caches are used to reduce database calls.

      This will provide performance improvements for restore and for activity duplication.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              tomotsuyuki Tomo Tsuyuki
              Reporter:
              mr-russ Russell Smith
              Peer reviewer:
              Mark Nelson Mark Nelson
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan, Matteo Scaramuccia, Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              3 Vote for this issue
              Watchers:
              15 Start watching this issue

                Dates

                Created:
                Updated: