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

Parallelize component loading/parsing #113

Open
seblund opened this issue Jul 28, 2022 · 0 comments
Open

Parallelize component loading/parsing #113

seblund opened this issue Jul 28, 2022 · 0 comments
Labels
approachable Easily approachable without taking major design decisions enhancement New feature or request

Comments

@seblund
Copy link
Member

seblund commented Jul 28, 2022

We might as well load and parse the individual components in parallel. Most of the code does not use Federations, so it is thread safe already. When #111 is complete we can revisit and parallelize even more code.

I suspect we can gain some performance improvements here, but we should benchmark the multithreaded implementation to ensure that it is indeed faster (the minor overhead may outweigh the benefits in some cases).

If we can parse/load components very quickly we may even be able to move to a stateless Reveaal where we maintain no state between queries. This allows us to do other parallelization more easily as there is no shared state between queries.

@seblund seblund added enhancement New feature or request approachable Easily approachable without taking major design decisions labels Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approachable Easily approachable without taking major design decisions enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant