-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
low_delivery queue v2 #30150
base: main
Are you sure you want to change the base?
low_delivery queue v2 #30150
Conversation
@@ -1,5 +1,5 @@ | |||
# frozen_string_literal: true | |||
|
|||
class ActivityPub::LowPriorityDeliveryWorker < ActivityPub::DeliveryWorker | |||
sidekiq_options queue: 'pull', retry: 8, dead: false | |||
sidekiq_options queue: 'low_delivery', retry: 8, dead: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may be over-analyzing, but I interpret low_delivery
as "this is something which is unlikely to be delivered". Would low_priority
or nonessential
or relaxed
or something (?) ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I originally named it for "Low priority delivery".
As I know, This sends these kind of activity
- Delete Actor
- Forward remotes' Delete activity
As it sends local "Delete Actor" activity, nonessential
doesn't reflect about it.
In my opinion, I'd like low_priority
Remake of #20216... please just close both PRs if this isn't worth the price.
Will require a lot of admins to add the additional queue.