composing dataloaders

Aperta
#54 22 commenti 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
30/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
graphql, java

Direzione di ricerca

Non è indicato alcun file sorgente né alcun test. Inizia tracciando il ciclo di vita di dispatchAll per l’esempio loader1.load(id1).thenCompose(...) del loader sequenziale, quindi esamina come dovrebbero essere eseguiti i caricamenti annidati. Il lavoro è completato quando i dataloader sequenziali terminano senza bloccarsi e il batching non richiede una chiamata a dispatch per ogni id.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

keep-open

Hi,

I have a datafetcher where I use 2 dataloaders in sequence: the first to translate from 1 ID to another, the second to fetch data corresponding to the second ID.

loader1.load(id1).thenCompose(id2 -> loader2.load(id2))

This hangs because dispatchAll() is not called again after loader1 completes.
I can work around that by adding that call inside the thenCompose() lambda but then it is called for every id2 which is ugly at the very least.

Is there a better way of doing this?

Lingua principale
Java
Stelle
525
Fork
101
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di graphql-java/java-dataloader

Tutte le issue di graphql-java/java-dataloader

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.