Skip to content

Rayon executor for GraphQL - #2128

Merged
ljeub-pometry merged 14 commits into
masterfrom
feature/rayon_exec
Jun 18, 2025
Merged

ljeub-pometry merged 14 commits into
masterfrom
feature/rayon_exec

Conversation

@ljeub-pometry

Copy link
Copy Markdown
Collaborator

What changes were proposed in this pull request?

Use rayon for compute-heavy blocking execution in raphtory-graphql

Why are the changes needed?

Should give better performance than using spawn-blocking

Does this PR introduce any user-facing change? If yes is this documented?

API is unchanged

How was this patch tested?

Are there any further changes required?

@ljeub-pometry
ljeub-pometry marked this pull request as ready for review June 18, 2025 09:03

@miratepuffin miratepuffin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment thread raphtory-graphql/src/data.rs
Comment thread raphtory-graphql/src/model/graph/edge.rs
@ljeub-pometry
ljeub-pometry merged commit 16e84ac into master Jun 18, 2025
@ljeub-pometry
ljeub-pometry deleted the feature/rayon_exec branch June 18, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants