Address comments #665

This commit is contained in:
Andras Slemmer
2017-05-11 11:32:29 +01:00
parent 8a17406246
commit b192a86a30
4 changed files with 8 additions and 8 deletions

View File

@ -63,11 +63,11 @@ class RPCStabilityTests {
startRpcClient<RPCOps>(server.get().hostAndPort).get() startRpcClient<RPCOps>(server.get().hostAndPort).get()
} }
} }
for (i in 1 .. 5) { repeat(5) {
startAndStop() startAndStop()
} }
val numberOfThreadsBefore = waitUntilNumberOfThreadsStable(executor) val numberOfThreadsBefore = waitUntilNumberOfThreadsStable(executor)
for (i in 1 .. 5) { repeat(5) {
startAndStop() startAndStop()
} }
val numberOfThreadsAfter = waitUntilNumberOfThreadsStable(executor) val numberOfThreadsAfter = waitUntilNumberOfThreadsStable(executor)
@ -87,11 +87,11 @@ class RPCStabilityTests {
ErrorOr.catch { startRpcClient<RPCOps>(server.get().hostAndPort, configuration = RPCClientConfiguration.default.copy(minimumServerProtocolVersion = 1)).get() } ErrorOr.catch { startRpcClient<RPCOps>(server.get().hostAndPort, configuration = RPCClientConfiguration.default.copy(minimumServerProtocolVersion = 1)).get() }
} }
} }
for (i in 1 .. 5) { repeat(5) {
startAndStop() startAndStop()
} }
val numberOfThreadsBefore = waitUntilNumberOfThreadsStable(executor) val numberOfThreadsBefore = waitUntilNumberOfThreadsStable(executor)
for (i in 1 .. 5) { repeat(5) {
startAndStop() startAndStop()
} }
val numberOfThreadsAfter = waitUntilNumberOfThreadsStable(executor) val numberOfThreadsAfter = waitUntilNumberOfThreadsStable(executor)

View File

@ -281,7 +281,7 @@ class RPCClientProxyHandler(
// leak borrowed executors. // leak borrowed executors.
val observationExecutors = observationExecutorPool.close() val observationExecutors = observationExecutorPool.close()
observationExecutors.forEach { it.shutdownNow() } observationExecutors.forEach { it.shutdownNow() }
lifeCycle.transition(State.FINISHED) lifeCycle.justTransition(State.FINISHED)
} }
/** /**

View File

@ -37,7 +37,7 @@ class LifeCycle<S : Enum<S>>(initial: S) {
} }
/** Transition the state to [to] without performing a current state check. */ /** Transition the state to [to] without performing a current state check. */
fun transition(to: S) { fun justTransition(to: S) {
lock.writeLock().withLock { lock.writeLock().withLock {
state = to state = to
} }

View File

@ -184,7 +184,7 @@ class RPCServer(
it.session.close() it.session.close()
it.sessionFactory.close() it.sessionFactory.close()
} }
lifeCycle.transition(State.FINISHED) lifeCycle.justTransition(State.FINISHED)
} }
private fun bindingRemovalArtemisMessageHandler(artemisMessage: ClientMessage) { private fun bindingRemovalArtemisMessageHandler(artemisMessage: ClientMessage) {