Merge pull request #503 from jglick/WorkflowRun.doStop

jglick · web-flow · commit 33a0c6f3a4b4 · 2025-01-17T14:18:54.000-05:00
Remove synchronization from `WorkflowRun.doStop`
diff --git a/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java b/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java
@@ -884,7 +884,7 @@ public boolean shouldCalculateCulprits() {
     }
 
     @RequirePOST
-    public synchronized HttpResponse doStop() {
+    public HttpResponse doStop() {
         Executor e = getOneOffExecutor();
         if (e != null) {
             return e.doStop();

Original file line number	Diff line number	Diff line change
`@@ -884,7 +884,7 @@ public boolean shouldCalculateCulprits() {`
`884`	`884`	`}`
`885`	`885`
`886`	`886`	`@RequirePOST`
`887`		`- public synchronized HttpResponse doStop() {`
	`887`	`+ public HttpResponse doStop() {`
`888`	`888`	`Executor e = getOneOffExecutor();`
`889`	`889`	`if (e != null) {`
`890`	`890`	`return e.doStop();`