Details

    • Type: Improvement Improvement
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.1.7, 2.2.5
    • Fix Version/s: Backlog
    • Component/s: Development, Performance
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • Similar issues:
      KULRICE-531development slowness
      KULRICE-1129page rendering slowness
      KULRICE-13070Pages load really slow after rice upgrade
      KULRICE-12988KC app startup time really slow with the latest rice 2.5 revision
      KULRICE-5548KRAD: IE is extremely slow to load pages
      KULRICE-5441Focus order & clean-up needed in Kuali portal header (inherited by all KNS & KS pages)
      KULRICE-12991Application startup time slow after Spring 4.0.x upgrade
      KULRICE-8849removePrincipalFromRole and getRoleMemberPrincipalId methods very slow
      KULRICE-568DD slowness fixes caused batch bugs and TP doc bugs (see comments)
      KULRICE-3818Make "all" label for drop-down for action list delegations more informative
    • Rice Module:
      KNS, KEW
    • KRAD Feature Area:
      Document
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      The superUserActions.tag file in rice performs some very expensive checks on every page display. Additionally, more checks than need to run even if the user is not authorized for various operations due to the fact that JSTL does not appear to short-circuit on evaluation of expressions.

      There are three main changes here:

      1) rearrange the checks in the tag file to prevent unnecessary changes.
      2) Wrap all the calls used from the JSP in transaction blocks so that individual connections and transactions are not opened for each database call.
      3) Eliminate the retrieval of the RouteNoteInstance objects before making the permission checks. The code in the KEW DocumentTypePermissionService only extracts the document number from the first returned result.

      1. KualiDocumentFormBase.java.diff
        11 kB
        Jonathan Keller
      2. superUserActions.tag.diff
        4 kB
        Jonathan Keller

        Activity

        Hide
        Jonathan Keller added a comment -

        Uploading patch files. Patch file was created from UCD current version and the Rice 2.1.7 base version.

        Show
        Jonathan Keller added a comment - Uploading patch files. Patch file was created from UCD current version and the Rice 2.1.7 base version.
        Hide
        Jonathan Keller added a comment -

        I have more fixes to upload here. I did further optimization on the calls in the form class to ensure only what was needed was being called.

        Show
        Jonathan Keller added a comment - I have more fixes to upload here. I did further optimization on the calls in the form class to ensure only what was needed was being called.

          People

          • Assignee:
            Unassigned
            Reporter:
            Jonathan Keller
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Structure Helper Panel