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

Grader report using way too much memory due to bad record counting

XMLWordPrintable

    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • MOODLE_27_STABLE
    • Hide

      1) create a course with a large number of users enrolled (say 10k)
      2) attempt to view the grader report
      Expected: report loads.
      Actual without this fix: page fails with out of memory error.

      Show
      1) create a course with a large number of users enrolled (say 10k) 2) attempt to view the grader report Expected: report loads. Actual without this fix: page fails with out of memory error.

      The get_numusers function in the grade_report class makes a call to get_enrolled_users asking for all the user fields (u.*), but then only makes use of the id field.

      Additionally, it makes the get_enrolled_users call regardless of if it actually needs that data or not. The call should only be made if the user has requested to only show active enrolments.

            aolley Adam Olley
            aolley Adam Olley
            Petr Skoda Petr Skoda
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Mark Nelson Mark Nelson
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.