- Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Description
I took a closer look at workqueueWithCustomAddFunc. It currently only overwrites the Add func. Would it also make sense to overwrite AddRateLimited, AddAfter and AddWithOpts?
For AddWithOpts ideally only set priority if not already set, but atm we can't differentiate between priority intentionally set to 0 and the zero value, because Priority in AddOpts is not a pointer (probably it should be).
I know that for our builtin event handler we only have to overwrite Add. But I assume if folks use WithLowPriorityWhenUnchanged they would probably expect that it works for all Add funcs.
Tasks:
- Consider to make AddOpts.Priority a pointer so we can differentiate between not set and priority 0 (the zero value)
Metadata
Metadata
Assignees
Labels
No labels