Skip to content
This repository has been archived by the owner on Jun 6, 2019. It is now read-only.

Handle messages concurrently #173

Open
adamralph opened this issue Oct 3, 2017 · 0 comments
Open

Handle messages concurrently #173

adamralph opened this issue Oct 3, 2017 · 0 comments
Labels

Comments

@adamralph
Copy link
Member

The V6 upgrade exposed the fact that the endpoints are currently not designed to handle messages concurrently.

Some of the problems:

  • The projectors (as currently designed) must update the views in serial to avoid a last-in-wins scenario while two messages are being concurrently handled.
  • The Twitter analyzer writes multiple lines to the console when handling each message, and these become interleaved when handling messages concurrently

There could be other issues too.

Note, as part of this, we should probably drop the use of ColoredConsole, since it does nasty locks around writing the console. We should be able to switch to using ANSI colour codes instead.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant