style: clarify workflow activity retry fields/defaults
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 62/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- python
調査の方向性
まず ext/dapr-ext-workflow/dapr/ext/workflow/retry_policy.py の RetryPolicy 定義付近から始め、フィールドとデフォルト値を、リンクされている durabletask-go workflow.go のリファレンスと比較します。フィールドの文書化された動作と妥当なデフォルト値を更新し、必要に応じてサポートされているフィールドを維持し、max_retry_interval を使用した retries を検証し、retry_timeout の失敗による結果を明確にします。
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- Python
- スター
- 272
- フォーク
- 152
- 平均マージ
- 1日 13時間
- マージ済み PR（30日）
- 4
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
dapr/python-sdk のほかの issue
-
dapr-ext-workflow good first issue kind/enhancement P2
難易度 2/5 1〜2日 初心者へのやさしさ 72/100
dapr/python-sdk#853 · コメント 4 件 ·
-
feature-request
難易度 3/5 1〜2日 初心者へのやさしさ 76/100
dapr/python-sdk#1214 ·
-
dapr-ext-workflow kind/enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 78/100
dapr/python-sdk#1213 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 76/100
dapr/python-sdk#1200 ·
-
dapr-ext-workflow kind/enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
dapr/python-sdk#1188 ·
dapr/python-sdk の issue をすべて見る
似ている issue
-
bug confirmed issue
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
open-webui/open-webui#30750 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
OpenwaterHealth/openmotion-bloodflow-app#604 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 90/100