Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Not version specific
    • Component/s: Development
    • Labels:
      None
    • Similar issues:
      KULRICE-2364Improve efficiency of duplicate rule check in RuleXmlParser
      KULRICE-1802Include Rule active indicator in duplicate check on Rule import
      KULRICE-8267Inefficiencies in persisting role members in the IdentityManagementRoleDocument (4062)
      KULRICE-2655Update RuleXmlParser to ensure it can still properly import rules and rule delegations
      KULRICE-667Make a proxy table to store service definitions in so we don't have to select them all regardless of service refresh etc.
      KULRICE-9019Like to have a way to fetch all valid TermSpecifications for a context
      KULRICE-13953IT Failure KewRuleDataJpaTest.testRuleDelegationServiceFindByDelegateRuleId Rule Delegation Bo fetched by rule id
      KULRICE-4790HideWebInfFilter inefficient
      KULRICE-2870All routing rules should have a unique Rule name, historical data should be updated
      KULRICE-13147Inefficiencies in view lifecycle and rendering
    • Rice Module:
      KEW

      Description

      Eric reported:

      In order to do a duplicate check, the RuleXmlParser.checkRuleForDuplicate method is doing the following:

      List rules = SpringServiceLocator.getRuleService().fetchAllRules(true);

      Which fetches every single rule in the system. This is an extremely slow operation (provided the rules aren't already cached). In addition to this it will cache every single one of these rules which may not be a sustainable model long term given that we can't gaurantee any sort of upper bound on the number of rules an installation site will have in their rules system.

      Sounds like we need a more efficient query in dao based on an example rule (although build a criteria object to query with)

        Activity

        Hide
        Aaron Godert (Inactive) added a comment -

        Originally reported by Ryan. Changing Reporter field so that he doesn't receive automatic notifications any further.

        Show
        Aaron Godert (Inactive) added a comment - Originally reported by Ryan. Changing Reporter field so that he doesn't receive automatic notifications any further.
        Hide
        Jessica Coltrin (Inactive) added a comment -

        This has been fixed along the way.

        Show
        Jessica Coltrin (Inactive) added a comment - This has been fixed along the way.

          People

          • Assignee:
            Unassigned
            Reporter:
            Aaron Godert (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel