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 created issue -
          Lukasz Karnasiewicz made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Lukasz Karnasiewicz made changes -
          Description Original: 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 at least a few times per day for
          every user. I don't have a thread dump available but I'll try to get it
          for reference.
          New: 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.
          Environment New: Ubuntu 14.04.1 LTS 64bit
          Jenkins ver. 1.625.2
          Java(TM) SE Runtime Environment (build 1.7.0_76-b13)
          Lukasz Karnasiewicz made changes -
          Link New: This issue is related to JENKINS-28292 [ JENKINS-28292 ]

          Lukasz Karnasiewicz added a comment - Fix: https://github.com/jenkinsci/jenkins/pull/1947
          Lukasz Karnasiewicz made changes -
          Status Original: In Progress [ 3 ] New: Open [ 1 ]
          Daniel Beck made changes -
          Labels New: regression

          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]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          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

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

              Created:
              Updated:
              Resolved: