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

Fix batch pass wrong arguments #49

Closed
wants to merge 3 commits into from
Closed

Fix batch pass wrong arguments #49

wants to merge 3 commits into from

Conversation

fcce
Copy link

@fcce fcce commented Nov 14, 2016

Superworker.define(:MyBatchSuperworker, :user_ids, :comment_ids) do
  batch user_ids: :user_id do
    Worker1 :user_id
  end
  Worker2 :comment_ids
end

MyBatchSuperworker.perform_async([10, 11, 12], [20, 21])

The above produces a sequence should equivalent to this :

Worker1.new.perform(10)
Worker1.new.perform(11)
Worker1.new.perform(12)
Worker2.new.perform([20,21])

@fcce
Copy link
Author

fcce commented Nov 14, 2016

also fix bug #39

@fcce fcce closed this Nov 20, 2016
@jeremyhaile
Copy link

@fcce I see this was closed - was it merged in?

@fcce
Copy link
Author

fcce commented Jul 7, 2017

sorry, not merged. This project seems don't updated any more. @jeremyhaile

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