Option retry_policy is dismissed on start_workflow.
まだ誰も着手していません。
評価
調査の方向性
lib/temporal/client.rb の43行目付近を調査し、start_workflow の実行オプションがどのように構築されているかを追跡してください。README に記載されている retry_policy が start_workflow_execution に渡されること、およびワークフローの開始時にそのオプションが機能することを確認してください。
索引モデルが issue の本文から書いたものです。
説明
I was trying to set a retry policy by passing a retry_policy hash in the options hash when calling start_workflow but it didn't work.
I went digging in the code and realized it is indeed omitted: even if the execution options are built with the policy, they are not used on the underlying call to start_workflow_execution at https://github.com/coinbase/temporal-ruby/blob/master/lib/temporal/client.rb#L43.
The only way I was able to actually set a policy was by calling retry_policy in an activity, but that leaves the caller powerless on setting a retry policy, and it seems it doesn't apply to workflows either.
Is this a regression? Was it always a bug? The readme states you can pass this option.
- 主要言語
- Ruby
- スター
- 288
- フォーク
- 113
- 平均マージ
- 10日 15時間
- マージ済み PR(30日)
- 2
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coinbase/temporal-ruby のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
coinbase/temporal-ruby#361 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
coinbase/temporal-ruby#341 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
coinbase/temporal-ruby#328 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
coinbase/temporal-ruby#326 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
coinbase/temporal-ruby#324 · コメント 3 件 · リアクション 2 件 ·
coinbase/temporal-ruby の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
riscv/riscv-unified-db#2626 ·
-
Component: GLib
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
ds-drift
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
we-promise/sure#3693 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
simp/pupmod-simp-simp#395 ·