broadinstitute / cromwell

Scientific workflow engine designed for simplicity & scalability. Trivially transition between one off use cases to massive scale production environments
http://cromwell.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
994 stars 360 forks source link

Should CromIamServer be using two actor systems? #3851

Open kshakir opened 6 years ago

kshakir commented 6 years ago

CromIamServer does not pass along its actor system so a new one is getting created by akka.http.scaladsl.server.HttpApp. Things seem stable at the moment so not touching it. Should investigate if we should be sharing the ActorSystem. If there is a reason then leave a comment why there should be two actor systems.

cjllanwarne commented 6 years ago

Relevant?

Any headline that ends in a question mark can be answered by the word no.