Skip to content

Commit 01e230f

Browse files
committed
Inline and rename
1 parent 8126e86 commit 01e230f

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

server/src/main/java/org/elasticsearch/cluster/coordination/PeerFinder.java

+6-10
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ public void activate(final DiscoveryNodes lastAcceptedNodes) {
9797
active = true;
9898
this.lastAcceptedNodes = lastAcceptedNodes;
9999
leader = Optional.empty();
100-
handleWakeUpUnderLock();
100+
handleWakeUp();
101101
}
102102
}
103103

@@ -106,7 +106,7 @@ public void deactivate(DiscoveryNode leader) {
106106
logger.trace("deactivating PeerFinder and setting leader to {}", leader);
107107
if (active) {
108108
active = false;
109-
handleWakeUpUnderLock();
109+
handleWakeUp();
110110
}
111111
this.leader = Optional.of(leader);
112112
assert assertInactiveWithNoKnownPeers();
@@ -159,12 +159,6 @@ public interface TransportAddressConnector {
159159
void connectToRemoteMasterNode(TransportAddress transportAddress, ActionListener<DiscoveryNode> listener);
160160
}
161161

162-
private void handleWakeUp() {
163-
synchronized (mutex) {
164-
handleWakeUpUnderLock();
165-
}
166-
}
167-
168162
private List<DiscoveryNode> getKnownPeers() {
169163
assert active;
170164
assert holdsLock() : "PeerFinder mutex not held";
@@ -184,7 +178,7 @@ private Peer createConnectingPeer(TransportAddress transportAddress) {
184178
return peer;
185179
}
186180

187-
private void handleWakeUpUnderLock() {
181+
private void handleWakeUp() {
188182
assert holdsLock() : "PeerFinder mutex not held";
189183

190184
for (final Peer peer : peersByAddress.values()) {
@@ -210,7 +204,9 @@ private void handleWakeUpUnderLock() {
210204
futureExecutor.schedule(new Runnable() {
211205
@Override
212206
public void run() {
213-
handleWakeUp();
207+
synchronized (mutex) {
208+
handleWakeUp();
209+
}
214210
}
215211

216212
@Override

0 commit comments

Comments
 (0)