Details

    • Type: Sub Task Sub Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: 1.0.1, KFS Release 3.0
    • Fix Version/s: 1.0.1.1, KFS Release 3.0.1
    • Component/s: Development
    • Labels:
      None
    • Similar issues:
      KULRICE-2673Optimize the RuleService.getDuplicateRuleId check
      KULRICE-7185Optimization of KIM Permission Checks
      KULRICE-8268broker pool optimization (4067)
      KULRICE-9432Optimize code in ViewHelperServiceImpl
      KULRICE-6757KSPD: Investigate Optimization options for ViewServiceImpl
      KULRICE-8844KualiDocumentFormBase permission checks are more expensive than they have to be
      KULRICE-14262Optimize Amazon RDS usage
      KULRICE-5956optimize site generation
      KULRICE-8329Optimize performance of collection totaling and grouping
      KULRICE-562Optimize file/directory scanning when loading DD files on startup
    • Rice Module:
      KEW
    • Application Requirement:
      Rice

      Description

      Reduce the number of times the full set of actions is retrieved from the database by loading the set once and using that for all the checks. Currently, each action being checked causes a full load of all action requests to see if the single action is in the list, and there is no caching on the retrieval.

      1. action_taken_optimization_partdeux.diff
        18 kB
        James Smith
      2. action_taken_optimization_partquatre.diff
        6 kB
        James Smith
      3. action_taken_optimization_parttrois.diff
        2 kB
        James Smith
      4. action_taken_optimization.patch
        18 kB
        Jonathan Keller
      1. Picture 3.png
        94 kB

        Issue Links

          Activity

          Hide
          Eric Westfall added a comment -

          Resolving this issue as a duplicate.

          Show
          Eric Westfall added a comment - Resolving this issue as a duplicate.
          Hide
          James Smith added a comment -

          Okay, here's the patch. With these changes, ValidActionsTest is fixed, so I'm going to commit, but I'm leaving this JIRA open so if you have further modifications, Jonathan, we track those. Thanks!

          Show
          James Smith added a comment - Okay, here's the patch. With these changes, ValidActionsTest is fixed, so I'm going to commit, but I'm leaving this JIRA open so if you have further modifications, Jonathan, we track those. Thanks!
          Hide
          James Smith added a comment -

          D'oh - by "leaving the JIRA open" I meant mine, KULRICE-3704.

          Show
          James Smith added a comment - D'oh - by "leaving the JIRA open" I meant mine, KULRICE-3704 .
          Hide
          James Smith added a comment -

          Added some extra code to keep the group ids for the principal around which should make things marginally more efficient.

          Show
          James Smith added a comment - Added some extra code to keep the group ids for the principal around which should make things marginally more efficient.
          Hide
          James Smith added a comment -

          No more patches after this, I promise.

          Show
          James Smith added a comment - No more patches after this, I promise.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel