style: clarify workflow activity retry fields/defaults
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 62/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- python
- Ambito
- distributed-systems
Direzione di ricerca
Inizia da ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py intorno alla definizione di RetryPolicy, quindi confronta i relativi campi e valori predefiniti con il riferimento collegato durabletask-go workflow.go. Aggiorna il comportamento documentato dei campi e i valori predefiniti sensati, mantieni i campi supportati come richiesto, verifica i retries con max_retry_interval e rendi esplicita la conseguenza di un errore di retry_timeout.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- Python
- Stelle
- 272
- Fork
- 152
- Merge medio
- 3g 21h
- PR unite (30g)
- 5
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di dapr/python-sdk
-
dapr-ext-workflow good first issue kind/enhancement P2
Difficoltà 2/5 1-2 giorni Idoneità per principianti 72/100
dapr/python-sdk#853 · 4 commenti ·
-
kind/bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
dapr/python-sdk#1230 ·
-
feature-request
Difficoltà 3/5 1-2 giorni Idoneità per principianti 76/100
dapr/python-sdk#1214 · 1 assegnatario ·
-
dapr-ext-workflow kind/enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 78/100
dapr/python-sdk#1213 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 76/100
dapr/python-sdk#1200 ·
Tutte le issue di dapr/python-sdk
Issue simili
-
area: harness bug status: needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Human-Agent-Society/reef#625 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 80/100
learningequality/kolibri#15351 · 2 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Name consistency Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
eellak/triplestore#65 · 1 commento ·