99designs / 99designs/cmdstalk
Performance: The run loop calls stats-job multiple times
- Vorherrschende Sprache
- Go
- Sterne
- 76
- Forks
- 14
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
For every loop of processing a job, there are multiple consecutive implicit calls to https://github.com/99designs/cmdstalk/blob/fbd704da919d283643e673bd02efcf0221224b92/bs/job.go#L93. This requires a full , unnecessary, roundtrip on the network.
To fix this, [the Run loop](https://github.com/99designs/cmdstalk/blob/fbd704da919d283643e673bd02efcf0221224b92/broker/broker.go#L97) could extract the stats for a job once, and parse the necessary stats.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Das Issue verweist auf broker/broker.go, Zeile 97, für die Run-Schleife und auf bs/job.go, Zeile 93, für den stats-job-Aufruf. Lies zunächst die Run-Methode des Brokers, um die Struktur der Schleife zu verstehen, und untersuche anschließend, wie Job-Statistiken abgerufen und verwendet werden. Die Behebung umfasst das Caching der Statistiken aus einem einzigen Aufruf und das Parsen innerhalb der Schleife. Überprüfe die vorhandenen Tests im broker-Paket, um sicherzustellen, dass die Änderungen keine bestehende Funktionalität beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Bereich
- backend, performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 50/100