Uploaded image for project: 'FishEye'
  1. FishEye
  2. FE-4125

Implicit committer mappings are potentially expensively computed multiple times on every request

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Low
    • 2.8.0, 2.8.0-m2
    • None
    • None

    Description

      CommitterUserMappingManager.getUserForCommitter() expensively calculates implicit mappings by calling userManager.getFishEyeUsers(). The result is cached, but this was still observed to account for 5.3% of rendering time of the Anon homepage on an AtlasEye demotion.

      Options to fix include any combination of the follow:

      1. On top of caching, implicit mappings should potentially be stored in the db.

      2. calculate implicit mappings at index time and store them.

      3. rewrite the implicit mapping searcher to use direct HQL to find a relevant user, rather than loading the entire user list.

      Attachments

        Activity

          People

            bhumphreys BrendanA
            bhumphreys BrendanA
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: