99designs / 99designs/cmdstalk

Performance: The run loop calls stats-job multiple times

Offen
#16 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Neue Issues direkt in Ihr Postfach

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