Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-16342

asynchPeople very slow when using Gravatar & Subversion plugins

    XMLWordPrintable

Details

    Description

      https://gist.github.com/4534378 shows https://jenkins.ci.cloudbees.com/job/libraries/asynchPeople/ taking an inordinate amount of time in the callback. Clearly AsynchPeople.data() cannot assume that UserAvatarResolver.resolve is quick. UserGravatarResolver sometimes blocks on an HTTP connection to a web service, though this is cached. However every time it checks Mailer.UserProperty.getAddress, which in the presence of SubversionMailAddressResolverImpl examines every build of every project in every folder in all of Jenkins. This is so inefficient we probably never want to do it, much less synchronously in an HTTP request.

      Besides People, the avatar is used in a couple of per-User pages, which is perhaps OK; but shown for every User in the HudsonPrivateSecurityRealm overview, which is definitely not OK.

      Attachments

        Issue Links

          Activity

            jglick Jesse Glick created issue -
            jglick Jesse Glick made changes -
            Field Original Value New Value
            Link This issue depends on JENKINS-15440 [ JENKINS-15440 ]
            jglick Jesse Glick made changes -
            Link This issue is blocking JENKINS-15206 [ JENKINS-15206 ]
            jglick Jesse Glick made changes -
            Priority Major [ 3 ] Critical [ 2 ]
            jglick Jesse Glick made changes -
            Link This issue depends on JENKINS-16390 [ JENKINS-16390 ]
            jglick Jesse Glick made changes -
            Link This issue depends on JENKINS-16437 [ JENKINS-16437 ]
            jglick Jesse Glick made changes -
            Link This issue depends on JENKINS-16390 [ JENKINS-16390 ]
            kutzi kutzi made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            jglick Jesse Glick made changes -
            Labels performance 1.480.3-fixed performance
            jglick Jesse Glick made changes -
            Link This issue depends on JENKINS-18118 [ JENKINS-18118 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 147160 ] JNJira + In-Review [ 192263 ]

            People

              jglick Jesse Glick
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: