Skip to content

Support Active Job Continuations #574

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 10, 2025
Merged

Conversation

djmb
Copy link
Contributor

@djmb djmb commented Jun 10, 2025

Active Job Continuations will be added in Rails 8.1. Queue adapters need to implement the stopping? method to support them.

Implement the method and set it to true when a worker is stopping.

Also add Rails versions so we can test against Rails main. To prevent an explosion of combinations, I've switched to only testing against minor versions of Ruby.

@djmb djmb force-pushed the rails-adapter-stopping branch from 3422a14 to 6bc62ee Compare June 10, 2025 13:55
[Active Job Continuations](rails/rails#55127)
will be added in Rails 8.1. Queue adapters need to implement the
`stopping?` method to support them.

Implement the method and set it to true when a worker is stopping.

Also add Rails versions so we can test against Rails main. To prevent an
explosion of combinations, I've switched to only testing against minor
versions of Ruby.
@djmb djmb force-pushed the rails-adapter-stopping branch from 6bc62ee to 7d02d7c Compare June 10, 2025 14:29
@djmb
Copy link
Contributor Author

djmb commented Jun 10, 2025

The failures against Rails main are due to a Rails bug - see rails/rails#55178.

Copy link
Member

@rosa rosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thank you so much @djmb 🙏

@rosa rosa merged commit 9bd2a1b into rails:main Jun 10, 2025
40 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants