Performance issues with ModelUtils._coerce and coerceArray
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- backend, performance
Direzione di ricerca
Inizia in lib/model-utils.js intorno alle righe 498-503, dove ModelUtils._coerce chiama coerceArray nel percorso try/catch. Esamina il comportamento di coerceArray e il carico di lavoro create-heavy descritto nell'issue, quindi verifica che la coercizione mantenga il comportamento esistente senza il costo ripetuto delle eccezioni e conferma il miglioramento tramite profiling.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
The way ModelUtils._coerce uses coerceArray is inefficient, and is causing it to show up as 20% of my application's CPU usage in profiler runs under a create heavy workload.
The problem code is this in juggler:
try {
// Coerce val into an array if it resembles an array-like object
val = coerceArray(val);
} catch (e) {
// NOOP when not coercable into an array.
}
Throwing and catching an error is much more expensive than returning a value, and nearly every call to _coerce hits this try/throw/catch path.

A variant of coerceArray that doesn't throw, but instead returns [arrayVal | undefined, coerced: boolean] I think would fix the performance problem here.
Logs
No response
Additional information
No response
Reproduction
design issue
- Lingua principale
- TypeScript
- Stelle
- 5.1k
- Fork
- 1.1k
- Merge medio
- 2g 16h
- PR unite (30g)
- 32
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di loopbackio/loopback-next
-
Docs good first issue help wanted
Difficoltà 1/5 1-3 ore Idoneità per principianti 68/100
loopbackio/loopback-next#683 · 3 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 55/100
loopbackio/loopback-next#11726 ·
-
bug
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
loopbackio/loopback-next#11500 · 1 commento · 1 reazione ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
loopbackio/loopback-next#11412 · 1 commento ·
-
bug help wanted
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
loopbackio/loopback-next#11396 · 2 commenti · 1 reazione ·
Tutte le issue di loopbackio/loopback-next
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
TanStack/tanstack.com#1293 ·