Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core: when running as worker, we only use a single thread (per worker) #9572

Closed
eckter opened this issue Nov 5, 2024 · 0 comments · Fixed by #9591
Closed

core: when running as worker, we only use a single thread (per worker) #9572

eckter opened this issue Nov 5, 2024 · 0 comments · Fixed by #9591
Assignees

Comments

@eckter
Copy link
Contributor

eckter commented Nov 5, 2024

The logs mention 16(?) threads when starting up, but we only use one.

To test this, the simplest way to test it is to add a Thread.sleep when processing messages (with lots of messages in the queue). It's easy to tell that the sleep freezes the whole thing, it doesn't process other messages in the meantime.

Ideally, we should enable proper multithreading with ways to configure it. It would be much faster when running the app locally, while still having a single infra loaded in memory.

Note: we may run into issues where several messages initiate an infra loading at the same time. InfraManager is supposed to be robust to this case though.

@eckter eckter changed the title core: when running as worker, we only use a single thread core: when running as worker, we only use a single thread (per worker) Nov 5, 2024
@eckter eckter self-assigned this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant