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

Performance regression after fixing synchronization issue when setting JDK installations. (issue 28292)

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core
    • Ubuntu 14.04.1 LTS 64bit
      Jenkins ver. 1.625.2
      Java(TM) SE Runtime Environment (build 1.7.0_76-b13)

      The fix for JENKINS-28292 included in 1.618 is causing performance
      issues - requests are hanging on Jenkins.getJDKs for a very long time
      (sometimes minutes). This does happen on 2 of our Jenkins instances and can be observed at least few times per day by different users. I don't have a thread dump available but I'll try to get it for reference.

          [JENKINS-31932] Performance regression after fixing synchronization issue when setting JDK installations. (issue 28292)

          Lukasz Karnasiewicz added a comment - Fix: https://github.com/jenkinsci/jenkins/pull/1947

          Code changed in jenkins
          User: Daniel Spilker
          Path:
          core/src/main/java/jenkins/model/Jenkins.java
          http://jenkins-ci.org/commit/jenkins/9a239081e945f9b1d3076bec59266d01f0857f47
          Log:
          avoid synchronization for Jenkinsget/setJDKs, use a volatile field instead

          [FIXES JENKINS-31932]

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Spilker Path: core/src/main/java/jenkins/model/Jenkins.java http://jenkins-ci.org/commit/jenkins/9a239081e945f9b1d3076bec59266d01f0857f47 Log: avoid synchronization for Jenkinsget/setJDKs, use a volatile field instead [FIXES JENKINS-31932]

          Code changed in jenkins
          User: Daniel Beck
          Path:
          core/src/main/java/jenkins/model/Jenkins.java
          http://jenkins-ci.org/commit/jenkins/0a5b03688ba6884429bb3d2a06be8b7955694420
          Log:
          Merge pull request #1971 from daspilker/JENKINS-31932

          JENKINS-31932 avoid synchronization for Jenkinsget/setJDKs

          Compare: https://github.com/jenkinsci/jenkins/compare/3f7163207fd7...0a5b03688ba6

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: core/src/main/java/jenkins/model/Jenkins.java http://jenkins-ci.org/commit/jenkins/0a5b03688ba6884429bb3d2a06be8b7955694420 Log: Merge pull request #1971 from daspilker/ JENKINS-31932 JENKINS-31932 avoid synchronization for Jenkinsget/setJDKs Compare: https://github.com/jenkinsci/jenkins/compare/3f7163207fd7...0a5b03688ba6

          Code changed in jenkins
          User: Daniel Beck
          Path:
          core/src/main/java/jenkins/model/Jenkins.java
          http://jenkins-ci.org/commit/jenkins/3be09ad1b4392a3a108c2c09abae3c554937b774
          Log:
          Merge pull request #1971 from daspilker/JENKINS-31932

          JENKINS-31932 avoid synchronization for Jenkinsget/setJDKs
          (cherry picked from commit 0a5b03688ba6884429bb3d2a06be8b7955694420)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: core/src/main/java/jenkins/model/Jenkins.java http://jenkins-ci.org/commit/jenkins/3be09ad1b4392a3a108c2c09abae3c554937b774 Log: Merge pull request #1971 from daspilker/ JENKINS-31932 JENKINS-31932 avoid synchronization for Jenkinsget/setJDKs (cherry picked from commit 0a5b03688ba6884429bb3d2a06be8b7955694420)

            lkarnasiewicz Lukasz Karnasiewicz
            lkarnasiewicz Lukasz Karnasiewicz
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: