Activity and Workflow lookup does not work with autoload
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 30/100
Research direction
Start at ExecutableLookup and trace how Temporal::Workflow and Temporal::Activity classes are cached during registration. Compare that path with the Rails.application.reloader.wrap middleware shown in the issue. Done means providing an optional development-time behavior that reflects changes to those classes after code reloads.
Written by the indexing model from the issue text.
Description
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.
- Dominant language
- Ruby
- Stars
- 287
- Forks
- 113
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from coinbase/temporal-ruby
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
coinbase/temporal-ruby#361 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
coinbase/temporal-ruby#341 ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
coinbase/temporal-ruby#328 · 1 comment ·
-
NoMethodError when await_workflow_result is called for long running workflowsPossibly taken @iekundayo claimed this 717 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 45/100
coinbase/temporal-ruby#326 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
coinbase/temporal-ruby#324 · 3 comments · 2 reactions ·
All issues in coinbase/temporal-ruby
Similar issues
-
data error
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
riscv/riscv-unified-db#2749 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 2 days
-
RubyGems
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
Maintainers usually reply within 1 day
-
Bug: Twelve Data per-minute throttle never engages (Rails.cache.read can't read the counter increment wrote on RedisCacheStore)Possibly taken @hharkins claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
we-promise/sure#4327 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
crmne/ruby_llm#1244 · 1 reaction ·
Maintainers usually reply within 1 day