Activity and Workflow lookup does not work with autoload
まだ誰も着手していません。
評価
調査の方向性
ExecutableLookup から始め、登録時に Temporal::Workflow クラスと Temporal::Activity クラスがどのようにキャッシュされるかを追跡します。issue に示されている Rails.application.reloader.wrap ミドルウェアとその経路を比較します。完了条件は、コードのリロード後にこれらのクラスへの変更を反映する、開発時に任意で有効化できる動作を提供することです。
索引モデルが issue の本文から書いたものです。
説明
I'm trying to implement Rails' autoloading feature in my local development so I can avoid restarting the workers every time I make a code change. This was easily achieved with a middleware
class ReloaderMiddleware
def call(_, &block)
::Rails.application.reloader.wrap(&block)
end
end
However, any Temporal::Workflow and Temporal::Activity class definition are cached in memory at registration time by ExecutableLookup, so changes in those particular classes are not reflected when the code reloads. It would be really nice to offer an option to reload those classes at run time, and make it optionally available only during development.
- 主要言語
- 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時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
googleapis/google-api-ruby-client#28001 · リアクション 5 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100