temporalio / temporalio/sdk-python
[Feature Request] Log and drop signals that pass the wrong number of arguments
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 1.2k
- Forks
- 241
- Ø Merge
- 3 T. 21 Std.
- Gemergte PRs (30 T.)
- 55
Beschreibung
Describe the solution you'd like
In #349 we now drop signals whose arguments are of the wrong type. But we still fail the workflow task if the arity is off, e.g. with Python exceptions like "missing 1 required positional argument". Try to make sure there is one argument for every required parameter and no more than accepted (varargs notwithstanding). May be easiest to have a min/max param count on the _SignalDefinition and just check against that.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Implementierung von _SignalDefinition und das in #349 eingeführte Verhalten zu lesen, und verfolge anschließend, wo Signalargumente validiert und aufgerufen werden. Als erledigt gilt die Aufgabe, wenn Signale mit zu wenigen oder zu vielen Argumenten protokolliert und verworfen werden, ohne dass die Workflow-Aufgabe fehlschlägt, während akzeptierte varargs weiterhin funktionieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100