synchronize checkLeadership

tisonkun · tisonkun · commit 0d4a7b75c34f · 2022-07-12T23:55:21.000+08:00
Signed-off-by: tison &lt;wander4096@gmail.com&gt;
diff --git a/curator-recipes/src/main/java/org/apache/curator/framework/recipes/leader/LeaderLatch.java b/curator-recipes/src/main/java/org/apache/curator/framework/recipes/leader/LeaderLatch.java
@@ -576,7 +576,7 @@ private synchronized void internalStart()
     @VisibleForTesting
     volatile CountDownLatch debugCheckLeaderShipLatch = null;
 
-    private void checkLeadership(List<String> children) throws Exception
+    private synchronized void checkLeadership(List<String> children) throws Exception
     {
         if ( debugCheckLeaderShipLatch != null )
         {

Original file line number	Diff line number	Diff line change
`@@ -576,7 +576,7 @@ private synchronized void internalStart()`
`576`	`576`	`@VisibleForTesting`
`577`	`577`	`volatile CountDownLatch debugCheckLeaderShipLatch = null;`
`578`	`578`
`579`		`- private void checkLeadership(List<String> children) throws Exception`
	`579`	`+ private synchronized void checkLeadership(List<String> children) throws Exception`
`580`	`580`	`{`
`581`	`581`	`if ( debugCheckLeaderShipLatch != null )`
`582`	`582`	`{`