Skip to content

Commit 0a80284

Browse files
committed
[grid] Fixing parameter order for NewSessionQueue [skip ci]
1 parent ba8dd31 commit 0a80284

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

java/server/src/org/openqa/selenium/grid/commands/Hub.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,8 @@ protected Handlers createHandlers(Config config) {
141141
NewSessionQueue sessionRequests = new LocalNewSessionQueue(
142142
tracer,
143143
bus,
144-
newSessionQueueOptions.getSessionRequestTimeout(),
145-
newSessionQueueOptions.getSessionRequestRetryInterval());
144+
newSessionQueueOptions.getSessionRequestRetryInterval(),
145+
newSessionQueueOptions.getSessionRequestTimeout());
146146
NewSessionQueuer queuer = new LocalNewSessionQueuer(tracer, bus, sessionRequests);
147147
handler.addHandler(queuer);
148148

java/server/src/org/openqa/selenium/grid/distributor/local/GridModel.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,12 +60,12 @@ public class GridModel {
6060
public GridModel(EventBus events) {
6161
this.events = Require.nonNull("Event bus", events);
6262

63-
events.addListener(NodeDrainStarted.listener(nodeId -> setAvailability(nodeId, DRAINING)));
64-
events.addListener(NodeDrainComplete.listener(this::remove));
65-
events.addListener(NodeRemovedEvent.listener(this::remove));
66-
events.addListener(NodeStatusEvent.listener(status -> refresh(status)));
63+
this.events.addListener(NodeDrainStarted.listener(nodeId -> setAvailability(nodeId, DRAINING)));
64+
this.events.addListener(NodeDrainComplete.listener(this::remove));
65+
this.events.addListener(NodeRemovedEvent.listener(this::remove));
66+
this.events.addListener(NodeStatusEvent.listener(this::refresh));
6767

68-
events.addListener(SessionClosedEvent.listener(this::release));
68+
this.events.addListener(SessionClosedEvent.listener(this::release));
6969
}
7070

7171
public GridModel add(NodeStatus node) {
@@ -188,7 +188,7 @@ public boolean reserve(SlotId slotId) {
188188

189189
if (!UP.equals(node.availability)) {
190190
LOG.warning(String.format(
191-
"Asked to reserve a slot on node %s, but not is %s",
191+
"Asked to reserve a slot on node %s, but node is %s",
192192
slotId.getOwningNodeId(),
193193
node.availability));
194194
return false;

0 commit comments

Comments
 (0)