cmr-exchange / dev-env-manager

An Alternate Development Environment Manager for the CMR
Apache License 2.0
0 stars 1 forks source link

Streamline async message logging #5

Closed oubiwann closed 6 years ago

oubiwann commented 6 years ago

We can use the pubsub component/capability once #4 is done.

oubiwann commented 6 years ago

Pubsub is now in place (as an implementation of inter-component messaging).

oubiwann commented 6 years ago

I think what should be done here is ...

  1. Create a new function in cmr.dev.env.manager.process.io called stream->pubsub
  2. Maybe convert cmr.dev.env.manager.process.io to a protocol and add configuration for selecting the default impl to use?
  3. Update cmr.dev.env.manager.process.core/spawn! to use the configured impl
  4. Update terminate! and terminate-external-process! to use the configured impl
oubiwann commented 6 years ago

Done (but not quite as planned; see 0b9f5c2 for details).