Jetspeed 2
  1. Jetspeed 2
  2. JS2-739

Improve Algorithms for Resource Validation and Template Localization

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.2
    • Fix Version/s: 2.1.2
    • Component/s: Decorations/Themes
    • Labels:
      None

      Description

      During load testing, we discovered the Decoration valve was calling code in the BaseDecorator that inefficiently made calls to validate resources, combined with applying a seldom used mediatype/language/countrycode fallback algorithm. This effort will optimize these inefficiencies with caching and optionalizing the fallback algorithm and resource validation. The overall expense of the existing algorithm was leading to as much as 18-24% CPU usage per request. Additionally, there are other calls made to locating templates like from velocity templates, where we may gain some more performance improvement.

        Issue Links

          Activity

          Hide
          David Sean Taylor added a comment -

          The decorator cache is in place. Contribution from myself and Woonsan Ko, a little pair programming across the Pacific
          It is disabled by default, as I believe the performance gains are not worthwhile until you get into heavy loads
          It now supports both JetspeedCache and session caching
          My opinion is that the entire Decoration code, and especially Content* classes, should be greatly improved on in the next release (2.2.)

          Show
          David Sean Taylor added a comment - The decorator cache is in place. Contribution from myself and Woonsan Ko, a little pair programming across the Pacific It is disabled by default, as I believe the performance gains are not worthwhile until you get into heavy loads It now supports both JetspeedCache and session caching My opinion is that the entire Decoration code, and especially Content* classes, should be greatly improved on in the next release (2.2.)

            People

            • Assignee:
              David Sean Taylor
              Reporter:
              David Sean Taylor
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development