style: clarify workflow activity retry fields/defaults
Maintainer antworten meist innerhalb von 7 Tagen
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 62/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- python
- Bereich
- distributed-systems
Rechercherichtung
Beginne mit ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py rund um die RetryPolicy-Definition und vergleiche anschließend ihre Felder und Defaults mit der verknüpften durabletask-go workflow.go-Referenz. Aktualisiere das dokumentierte Verhalten der Felder und sinnvolle Defaults, behalte unterstützte Felder wie erforderlich bei, überprüfe Retries mit max_retry_interval und mache die Konsequenz eines retry_timeout-Fehlers explizit.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the WORKFLOW SDK feature
There are multiple gotchas in the workflow activity retry policies as described below:
ACTIVITY_RETRY_POLICY = RetryPolicy(
first_retry_interval=timedelta(seconds=1),
# max_number_of_attempts is the total attempt count, not the retry count
max_number_of_attempts=5,
backoff_coefficient=3,
# Set max_retry_interval else there won't be any retries
max_retry_interval=timedelta(seconds=10),
# DON'T set retry_timeout. If the timeout is hit, the entire workflow fails and the
# final activity isn't executed.
)
https://github.com/dapr/python-sdk/blob/main/ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py#L26
go-sdk ref https://github.com/dapr/durabletask-go/blob/main/workflow/workflow.go#L147
acceptance criteria:
- update the field values to sensible defaults if not provided
- max_number_of_attempts can remain as is, or become deprecated (still supported) and a new field of max_attempts added that will eventuallyyyyy replace this field
- all field docs strings updated to be clearer on what they do
- ensure that retries still apply if max_retry_interval is set
- update retry_timeout to indicate that if this fails then the workflow will fail
Release Note
RELEASE NOTE: ADD Clarify workflow activity retry policies fields and defaults
- Vorherrschende Sprache
- Python
- Sterne
- 274
- Forks
- 157
- Ø Merge
- 8 T. 20 Std.
- Gemergte PRs (30 T.)
- 31
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine 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.
Mehr aus dapr/python-sdk
-
Report the http(s):// gRPC endpoint deprecation as a DeprecationWarning, not a UserWarning on every clientEvtl. vergeben @kocaemre hat das vor 1 Tag übernommen. Offengood first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
dapr/python-sdk#1275 ·
Maintainer antworten meist innerhalb von 7 Tagen
-
SDK Alignment of Workflow ExamplesEvtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offendapr-ext-workflow good first issue kind/enhancement P2
Schwierigkeit 2/5 1-2 Tage Anfängerfreundlichkeit 72/100
dapr/python-sdk#853 · 4 Kommentare ·
Maintainer antworten meist innerhalb von 7 Tagen
-
DAPR_* environment variables are snapshotted at import time, so loading config after import silently dials localhostEvtl. vergeben @CasperGN hat das vor 5 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
dapr/python-sdk#1259 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 7 Tagen
-
[WORKFLOW SDK FEATURE REQUEST] Retry WaitForInstanceCompletion/Start on a server-sent CANCELLEDOffendapr-ext-workflow kind/enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
dapr/python-sdk#1246 ·
Maintainer antworten meist innerhalb von 7 Tagen
-
[BUG] subscribe_with_handler stops silently: async task can die or be collected, sync retry path exits, handler errors reconnect the streamEvtl. vergeben @CasperGN hat das vor 20 Tagen übernommen. Offenkind/bug
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 65/100
dapr/python-sdk#1233 ·
Maintainer antworten meist innerhalb von 7 Tagen
Alle Issues in dapr/python-sdk
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
WingedGuardian/GENesis-AGI#3375 ·
Maintainer antworten meist innerhalb von 1 Tag
-
curriculum documentation quality
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
githubnext/gh-aw-workshop#4601 ·
Maintainer antworten meist innerhalb von 1 Tag
-
approved Good First Issue scope: docs
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
hiero-ledger/hiero-sdk-python#2730 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag