From 182e99c24381fe1c7d3f2ec3f47d0b579b774f60 Mon Sep 17 00:00:00 2001
From: Jesse Glick <jglick@cloudbees.com>
Date: Mon, 2 Nov 2015 09:46:25 -0500
Subject: [PATCH] @oleg-nenashev suggests making get/setBuildDiscarder
 synchronized.

---
 core/src/main/java/hudson/model/Job.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/hudson/model/Job.java b/core/src/main/java/hudson/model/Job.java
index 767329db2c..07684f9e9d 100644
--- a/core/src/main/java/hudson/model/Job.java
+++ b/core/src/main/java/hudson/model/Job.java
@@ -429,12 +429,12 @@ public abstract class Job<JobT extends Job<JobT, RunT>, RunT extends Run<JobT, R
     /**
      * Returns the configured build discarder for this job, via {@link BuildDiscarderProperty}, or null if none.
      */
-    public BuildDiscarder getBuildDiscarder() {
+    public synchronized BuildDiscarder getBuildDiscarder() {
         BuildDiscarderProperty prop = getProperty(BuildDiscarderProperty.class);
         return prop != null ? prop.getStrategy() : /* settings compatibility */ logRotator;
     }
 
-    public void setBuildDiscarder(BuildDiscarder bd) throws IOException {
+    public synchronized void setBuildDiscarder(BuildDiscarder bd) throws IOException {
         BulkChange bc = new BulkChange(this);
         try {
             removeProperty(BuildDiscarderProperty.class);
-- 
GitLab