style: clarify workflow activity retry fields/defaults
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 62/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- python
- Lĩnh vực
- distributed-systems
Hướng nghiên cứu
Bắt đầu với ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py, tại phần định nghĩa RetryPolicy, sau đó so sánh các trường và giá trị mặc định của nó với tài liệu tham chiếu durabletask-go workflow.go được liên kết. Cập nhật hành vi của các trường được mô tả trong tài liệu và các giá trị mặc định hợp lý, giữ lại các trường được hỗ trợ khi cần, xác minh retries với max_retry_interval và nêu rõ hậu quả của lỗi retry_timeout.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- Python
- Star
- 272
- Fork
- 152
- Merge trung bình
- 3 ngày 21 giờ
- Pull request đã merge (30 ngày)
- 5
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của dapr/python-sdk
-
dapr-ext-workflow good first issue kind/enhancement P2
Độ khó 2/5 1-2 ngày Mức phù hợp với người mới 72/100
dapr/python-sdk#853 · 4 bình luận ·
-
kind/bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
dapr/python-sdk#1230 ·
-
feature-request
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 76/100
dapr/python-sdk#1214 · 1 người được giao ·
-
dapr-ext-workflow kind/enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 78/100
dapr/python-sdk#1213 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 76/100
dapr/python-sdk#1200 ·
Tất cả issue của dapr/python-sdk
Issue tương tự
-
[Bug] reef-hermes tells me to resume with hermes --resume, which does not work from my shell Đang mởarea: harness bug status: needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Human-Agent-Society/reef#625 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 80/100
learningequality/kolibri#15351 · 2 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Name consistency Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
eellak/triplestore#65 · 1 bình luận ·