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

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open In Progress In Progress
          5s 1 David Sean Taylor 22/Jun/07 17:01
          In Progress In Progress Open Open
          12d 4h 52m 1 David Sean Taylor 04/Jul/07 21:53
          Open Open Resolved Resolved
          7m 7s 1 David Sean Taylor 04/Jul/07 22:00
          Resolved Resolved Closed Closed
          1552d 23h 36m 1 Ate Douma 04/Oct/11 21:37
          Ate Douma made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Dennis Dam made changes -
          Link This issue relates to JS2-867 [ JS2-867 ]
          David Sean Taylor made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          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.)
          David Sean Taylor made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          David Sean Taylor made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]
          David Sean Taylor created issue -

            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