Skip to content
Snippets Groups Projects
Unverified Commit 9a4116cc authored by Juan Ignacio Ubeira's avatar Juan Ignacio Ubeira Committed by GitHub
Browse files

Merge pull request #296 from rjcausarano/test-fix-proposal

Fix problems shutting down nodes
parents 1f2ec3ea 18241878
No related branches found
No related tags found
No related merge requests found
...@@ -9,3 +9,4 @@ bin ...@@ -9,3 +9,4 @@ bin
*.iml *.iml
.idea .idea
bazel-* bazel-*
local.properties
...@@ -416,8 +416,6 @@ public class DefaultNode implements ConnectedNode { ...@@ -416,8 +416,6 @@ public class DefaultNode implements ConnectedNode {
// NOTE(damonkohler): We don't want to raise potentially spurious // NOTE(damonkohler): We don't want to raise potentially spurious
// exceptions during shutdown that would interrupt the process. This is // exceptions during shutdown that would interrupt the process. This is
// simply best effort cleanup. // simply best effort cleanup.
slaveServer.shutdown();
topicParticipantManager.shutdown();
for (ServiceServer<?, ?> serviceServer : serviceManager.getServers()) { for (ServiceServer<?, ?> serviceServer : serviceManager.getServers()) {
try { try {
Response<Integer> response = Response<Integer> response =
...@@ -436,8 +434,9 @@ public class DefaultNode implements ConnectedNode { ...@@ -436,8 +434,9 @@ public class DefaultNode implements ConnectedNode {
for (ServiceClient<?, ?> serviceClient : serviceManager.getClients()) { for (ServiceClient<?, ?> serviceClient : serviceManager.getClients()) {
serviceClient.shutdown(); serviceClient.shutdown();
} }
registrar.shutdown();
slaveServer.shutdown(); slaveServer.shutdown();
topicParticipantManager.shutdown();
registrar.shutdown();
signalOnShutdownComplete(); signalOnShutdownComplete();
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment