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

[proposal] Add @concurrent directive for types #3202

Open
krupyansky opened this issue Jul 30, 2024 · 0 comments
Open

[proposal] Add @concurrent directive for types #3202

krupyansky opened this issue Jul 30, 2024 · 0 comments

Comments

@krupyansky
Copy link
Contributor

What happened?

For every element of list on marshaling graphql object we get one goroutine. It's bad for performance on load tests. We should use goroutine for network call or another system call. Or hard parallel calculations. By marshaling isn't hard calculation.

What did you expect?

I propose to add "@Concurrent" directive for control the behavior. That lists consists concurrent type will have generated code as is now. But for lists with not concurrent type (without @Concurrent directive) will not have concurrent code.

@krupyansky krupyansky changed the title Add @concurrent directive for types [proposal] Add @concurrent directive for types Jul 30, 2024
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

No branches or pull requests

1 participant