Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-39895

Excessive Sakai user login times causing user frustration

    XMLWordPrintable

    Details

    • Type: (Deprecated) Contributed Patch
    • Status: CLOSED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.6.x
    • Fix Version/s: 2.8.x, 2.9.x
    • Component/s: Kernel
    • Labels:
      None
    • Environment:
      Sakai 2.6.2
    • CLE Team Issue:
      Yes
    • Previous Issue Keys:
      KNL-603

      Description

      This patch is contributed by the University of Virginia and addresses a performance issue related to Sakai login times.

      As the amount of CM data has increased over time, Sakai login times were becoming longer and longer. This performance issue has been successfully addressed for us with assistance from the email from Lydia Li at Stanford: http://article.gmane.org/gmane.comp.cms.sakai.devel/36245. The problem was that realms were being refreshed on every login which gets slower and slower as the user has more and more CM data. These realms do not need to be refreshed in our environment since our CMSync job (which runs daily) refreshes all of the realms and the CM data is only imported once per day from our SIS.

      The fix suggested by Lydia involves a change to kernel-impl/src/main/java/org/sakaiproject/authz/impl/BaseAuthzGroupService.java in the refreshUser() method.

      In addition to the attached patch file for the kernel modification, the UVa fix incorporates a new property. The 'suppressCMRefresh' property allows users to enable the fix, or not, as it suits their installation. This property needs to be included in 'sakai.properties':

      #

      1. login acceleration: 'true' means do not refresh realm tables; 'false' follows the default behavior
        #
        suppressCMRefresh=true
        #

      This patch is contributed for consideration with the hopes it will be incorporated into a future Sakai release.

        Gliffy Diagrams

          Zeplin

            Attachments

              Activity

                People

                Assignee:
                dhorwitz David Horwitz
                Reporter:
                cmw6s Cynthia Gast
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    Git Integration