99designs / 99designs/gqlgen

[proposal] Add @concurrent directive for types

Offen
#3,202 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
10.8k
Forks
1.3k
Ø Merge
2 T. 36 Min.
Gemergte PRs (30 T.)
26

Beschreibung

### 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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.