3 Replies Latest reply on Mar 19, 2008 10:16 AM by alesj

    Locking and the DeployersImpl.shutdown()

      Related to this:

      There's some code/test I'm going break
      relating to shutdown and single deployment.

      It was DeployerSingleDeploymentTestCase::testMultiThreadsAndShutdown()

      This is incorrect behaviour.

      The shutdown should not be blocked by a deployment, it were
      it could "deadlock" the shutdown when the deployment is misbehaving (e.g. a broken service
      that is waiting on a socket forever because of a network split)

      I'm going to remove this test.

      The test actually suffers from a race anyway.
      The deployment threads (as written currently) could run after the shutdown thread
      gets the lock which means the test would fail.