This is an automated email from the ASF dual-hosted git repository. klund pushed a commit to branch support/1.13 in repository https://gitbox.apache.org/repos/asf/geode.git
commit fd664de8a00dfc9b61bf54d869129a227889dcdd Author: Kirk Lund <kl...@apache.org> AuthorDate: Tue Jan 12 09:55:05 2021 -0800 GEODE-8696: Remove redundant syncs to prevent deadlock (#5894) FederatingManager now uses a ReentrantLock instead of synchronization. (cherry picked from commit 0e3ade53f724990a0932151a6fb6e90fe510758f) --- .../org/apache/geode/management/internal/FederatingManager.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/FederatingManager.java b/geode-core/src/main/java/org/apache/geode/management/internal/FederatingManager.java index 3ba7c21..afc51d4 100755 --- a/geode-core/src/main/java/org/apache/geode/management/internal/FederatingManager.java +++ b/geode-core/src/main/java/org/apache/geode/management/internal/FederatingManager.java @@ -112,7 +112,7 @@ public class FederatingManager extends Manager implements ManagerMembership { * Management exception has to be handled by the caller. */ @Override - public synchronized void startManager() { + public void startManager() { try { lifecycleLock.lock(); try { @@ -162,7 +162,7 @@ public class FederatingManager extends Manager implements ManagerMembership { } @Override - public synchronized void stopManager() { + public void stopManager() { lifecycleLock.lock(); try { // remove hidden management regions and federatedMBeans @@ -352,7 +352,7 @@ public class FederatingManager extends Manager implements ManagerMembership { } } - private synchronized void executeTask(Runnable task) { + private void executeTask(Runnable task) { try { executorService.get().execute(task); } catch (RejectedExecutionException ignored) {