discourse/app/jobs/scheduled/enqueue_digest_emails.rb
Sam 120fa8ad2f
PERF: Introduce absolute limit of digests per 30 minutes (#10845)
To avoid blocking the sidekiq queue a limit of 10,000 digests per 30 minutes
is introduced.

This acts as a safety measure that makes sure we don't keep pouring oil on
a fire.

On multisites it is recommended to set the number way lower so sites do not
dominate the backlog. A reasonable default for multisites may be 100-500.

This can be controlled with the environment var

DISCOURSE_MAX_DIGESTS_ENQUEUED_PER_30_MINS_PER_SITE
2020-10-07 17:30:15 +11:00

45 lines
1.6 KiB
Ruby

# frozen_string_literal: true
module Jobs
class EnqueueDigestEmails < ::Jobs::Scheduled
every 30.minutes
def execute(args)
return if SiteSetting.disable_digest_emails? || SiteSetting.private_email?
users = target_user_ids
if users.length > GlobalSetting.max_digests_enqueued_per_30_mins_per_site
users = users.shuffle[0...GlobalSetting.max_digests_enqueued_per_30_mins_per_site]
end
users.each do |user_id|
::Jobs.enqueue(:user_email, type: :digest, user_id: user_id)
end
end
def target_user_ids
# Users who want to receive digest email within their chosen digest email frequency
query = User
.real
.activated
.not_suspended
.where(staged: false)
.joins(:user_option, :user_stat, :user_emails)
.where("user_options.email_digests")
.where("user_stats.bounce_score < #{SiteSetting.bounce_score_threshold}")
.where("user_emails.primary")
.where("COALESCE(last_emailed_at, '2010-01-01') <= CURRENT_TIMESTAMP - ('1 MINUTE'::INTERVAL * user_options.digest_after_minutes)")
.where("COALESCE(last_seen_at, '2010-01-01') <= CURRENT_TIMESTAMP - ('1 MINUTE'::INTERVAL * user_options.digest_after_minutes)")
.where("COALESCE(last_seen_at, '2010-01-01') >= CURRENT_TIMESTAMP - ('1 DAY'::INTERVAL * #{SiteSetting.suppress_digest_email_after_days})")
# If the site requires approval, make sure the user is approved
query = query.where("approved OR moderator OR admin") if SiteSetting.must_approve_users?
query.pluck(:id)
end
end
end