style: clarify workflow activity retry fields/defaults
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 62/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- distributed-systems
Línea de trabajo
Comienza con ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py alrededor de la definición de RetryPolicy y, a continuación, compara sus campos y valores predeterminados con la referencia vinculada durabletask-go workflow.go. Actualiza el comportamiento documentado de los campos y los valores predeterminados sensatos, conserva los campos compatibles según sea necesario, verifica los reintentos con max_retry_interval y haz explícita la consecuencia de un fallo de retry_timeout.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Python
- Estrellas
- 272
- Forks
- 152
- Merge medio
- 3 d 21 h
- PR fusionados (30 d)
- 5
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de dapr/python-sdk
-
dapr-ext-workflow good first issue kind/enhancement P2
Dificultad 2/5 1-2 días Aptitud para principiantes 72/100
dapr/python-sdk#853 · 4 comentarios ·
-
kind/bug
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
dapr/python-sdk#1230 ·
-
feature-request
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
dapr/python-sdk#1214 · 1 asignado ·
-
dapr-ext-workflow kind/enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 78/100
dapr/python-sdk#1213 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
dapr/python-sdk#1200 ·
Todos los issues de dapr/python-sdk
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
xinnan-tech/xiaozhi-fde-talk#263 ·
-
rules
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
huggingface/Repo2RLEnv#163 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
huggingface/sentence-transformers#4074 ·
-
comp/dashboard invalid P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
NousResearch/hermes-agent#121143 ·