Repository navigation
[JENKINS-31932] Performance regression after fixing synchronization issue when setting JDK installations. (issue 28292) #21624
Description
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.
Originally reported by lkarnasiewicz, imported from: Performance regression after fixing synchronization issue when setting JDK installations. (issue 28292)
- assignee: lkarnasiewicz
- status: Resolved
- priority: Major
- component(s): core
- label(s): 1.642.2-fixed, regression
- resolution: Fixed
- resolved: 2016-01-08T12:34:13+00:00
- votes: 1
- watchers: 4
- imported: 2025-11-24
Raw content of original issue
The fix for
JENKINS-28292included 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
Ubuntu 14.04.1 LTS 64bit<br/>
Jenkins ver. 1.625.2<br/>
Java(TM) SE Runtime Environment (build 1.7.0_76-b13)
Activity
jenkins-infra-bot commentedon Dec 7, 2015
lkarnasiewicz:
Raw content of original comment:
Fix: #1947
jenkins-infra-bot commentedon Jan 8, 2016
Raw content of original 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]
jenkins-infra-bot commentedon Jan 8, 2016
Raw content of original 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-31932JENKINS-31932avoid synchronization for Jenkinsget/setJDKsCompare: 3f71632...0a5b036
jenkins-infra-bot commentedon Feb 3, 2016
Raw content of original 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-31932JENKINS-31932avoid synchronization for Jenkinsget/setJDKs(cherry picked from commit 0a5b036)
jenkins-infra-bot commentedon Nov 24, 2025
[Original
relates_tofrom Jira: JENKINS-28292]