Skip to content
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

Support bulk enqueue with differing class and options #420

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dtcristo
Copy link
Contributor

@dtcristo dtcristo commented Mar 4, 2024

Follow-up to #340.

Allow bulk enqueue of multiple different job classes and differing job options in a single .bulk_enqueue block. Each job can now differ by job class, queue, priority, run at and tags (in addition to args and kwargs).

@dtcristo dtcristo force-pushed the dtcristo/bulk-enqueue-improvements branch from 771aba0 to 6a803fa Compare March 5, 2024 00:42
@dtcristo
Copy link
Contributor Author

dtcristo commented Mar 5, 2024

I rebased this PR on top of #421 as CI was broken due to Bundler version.

Copy link
Member

@ZimbiX ZimbiX left a comment

Choose a reason for hiding this comment

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

Great work! Thanks for this :D

lib/que/job.rb Show resolved Hide resolved
lib/que/job.rb Outdated Show resolved Hide resolved
spec/que/job.bulk_enqueue_spec.rb Outdated Show resolved Hide resolved
spec/que/job.bulk_enqueue_spec.rb Show resolved Hide resolved
[Bundler 2.5 dropped support for Ruby 2.7](https://github.com/rubygems/rubygems/releases/tag/bundler-v2.5.0).
Since we are testing with Ruby 2.7, we should pin the version of Bundler
to 2.4.x since this is the last version that's compatible with 2.7.
Prepare spec for bulk enqueue refactor to support bulk enqueue of jobs
with differing job class and job options.
@dtcristo dtcristo force-pushed the dtcristo/bulk-enqueue-improvements branch from 6a803fa to 63f5817 Compare March 5, 2024 23:44
Allow bulk enqueue of multiple different job classes and differing job
options in a single `.bulk_enqueue` block. Each job can now differ by
job class, queue, priority, run at and tags (in addition to args and
kwargs).
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