about summary refs log tree commit diff
path: root/app/workers
diff options
context:
space:
mode:
authortakayamaki <fsgiko@gmail.com>2017-06-04 07:11:15 +0900
committerEugen Rochko <eugen@zeonfederated.com>2017-06-04 00:11:15 +0200
commit3eedad27375721aaaa6e8397124989e4ead771a9 (patch)
tree08d44ddfd6c217b9311efad92daf55328568cc82 /app/workers
parentce7c0def88b4f9ee97913002fb2d41d1832bd8af (diff)
change sidekiq queueing to bulk push (#3536)
Diffstat (limited to 'app/workers')
-rw-r--r--app/workers/scheduler/subscriptions_scheduler.rb6
-rw-r--r--app/workers/soft_block_domain_followers_worker.rb7
2 files changed, 8 insertions, 5 deletions
diff --git a/app/workers/scheduler/subscriptions_scheduler.rb b/app/workers/scheduler/subscriptions_scheduler.rb
index 3ea3ad2b8..7bfd002f4 100644
--- a/app/workers/scheduler/subscriptions_scheduler.rb
+++ b/app/workers/scheduler/subscriptions_scheduler.rb
@@ -1,5 +1,7 @@
 # frozen_string_literal: true
+
 require 'sidekiq-scheduler'
+require 'sidekiq-bulk'
 
 class Scheduler::SubscriptionsScheduler
   include Sidekiq::Worker
@@ -7,9 +9,7 @@ class Scheduler::SubscriptionsScheduler
   def perform
     logger.info 'Queueing PuSH re-subscriptions'
 
-    expiring_accounts.pluck(:id).each do |id|
-      Pubsubhubbub::SubscribeWorker.perform_async(id)
-    end
+    Pubsubhubbub::SubscribeWorker.push_bulk(expiring_accounts.pluck(:id))
   end
 
   private
diff --git a/app/workers/soft_block_domain_followers_worker.rb b/app/workers/soft_block_domain_followers_worker.rb
index 2782d05d2..ce76683c5 100644
--- a/app/workers/soft_block_domain_followers_worker.rb
+++ b/app/workers/soft_block_domain_followers_worker.rb
@@ -1,13 +1,16 @@
 # frozen_string_literal: true
 
+require 'sidekiq-bulk'
+
 class SoftBlockDomainFollowersWorker
   include Sidekiq::Worker
 
   sidekiq_options queue: 'pull'
 
   def perform(account_id, domain)
-    Account.find(account_id).followers.where(domain: domain).pluck(:id).each do |follower_id|
-      SoftBlockWorker.perform_async(account_id, follower_id)
+    followers_id = Account.find(account_id).followers.where(domain: domain).pluck(:id)
+    SoftBlockWorker.push_bulk(followers_id) do |follower_id|
+      [account_id, follower_id]
     end
   end
 end