Allow Bulk-enqueueing jobs with concurrency control?

[Timmitry]

I wanted to use bulk enqueueing of jobs in a project. It worked with both approaches (block form or instantiating jobs and then using GoodJob::Bulk.enqueue(jobs)), however, when looking at the SQL logs, I saw one insert statement for each job instead of just one in total.

Digging deeper, I found the relevant code which prevents that:

good_job/lib/good_job/bulk.rb

Lines 95 to 98 in 0ffbbac

	unbulkable_jobs, bulkable_jobs = jobs.partition do |job|
	job.respond_to?(:good_job_concurrency_key) && job.good_job_concurrency_key &&
	(job.class.good_job_concurrency_config[:enqueue_limit] || job.class.good_job_concurrency_config[:total_limit])
	end

So jobs are partitioned into bulkable and unbulkable jobs - and jobs are unbulkable if they have concurrency control configured. I understand why this could be a problem with batch-insertion of jobs, but still wanted to check whether implementing this could be possible. Otherwise, it might at least be good to mention that limitation in the readme (I'll gladly contribute that!).

At least for me, this unfortunately makes bulk unusable, as I alway configure concurrency for all jobs as a best practice 🤪 But aside of that, I totally love this gem ❤️