The task hash stores a queue_name field that no reader uses
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 72/100
Rechercherichtung
Beginne in threadmill/backends/redis.py bei der Enqueue-Hash-Zuordnung und untersuche anschließend die acquire- und mover-Skripte sowie das im Issue beschriebene Verhalten des reaper. Entscheide, ob das ungenutzte Feld queue_name entfernt werden sollte oder ob ein Reader es verarbeiten sollte; abgeschlossen ist die Arbeit, wenn der Hash kein nicht verarbeitetes Feld mehr schreibt und das bestehende queue-name-Verhalten erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Found while reviewing PR #69 with the SuperJoe crew. Filed as out-of-scope there.
RedisTaskBackend.enqueue() stores queue_name in the task hash. No reader uses it. The acquire and mover scripts read data and score. The reaper only makes sure that the hash exists. Every reader takes the queue from task.queue_name in the payload or from the key that holds the id. The backend writes this field on each enqueue, but no code reads it.
Remove the field, or make one reader use it. Then the payload does not need to carry the queue name as well.
Reference: threadmill/backends/redis.py, the enqueue hash mapping.
- Vorherrschende Sprache
- Python
- Sterne
- 21
- Forks
- 2
- Ø Merge
- 12 Std. 39 Min.
- Gemergte PRs (30 T.)
- 27
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Ähnliche Issues
-
Edit tool converts CRLF files to LFEvtl. vergeben @RaphaelFakhri hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
huggingface/tau#774 ·
Maintainer antworten meist innerhalb von 1 Tag
-
show ignores the row limit when a column or table name contains "limit"Evtl. vergeben @RaphaelFakhri hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
Maintainer antworten meist innerhalb von 2 Tagen
-
documentation good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
eclipse-score/inc_someip_gateway#356 ·
Maintainer antworten meist innerhalb von 2 Tagen