-
Notifications
You must be signed in to change notification settings - Fork 248
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
Migrate queue's queue to a more visible form #2940
Comments
Note that postgres supports NOTIFY/LISTEN, allowing us to avoid polling (or for safety's sake, polling very infrequently) the DB when nothing is changing. |
current thought progress documented in this gist https://gist.github.com/lotas/57779afe067e4f8eb586f37fe25016c9 |
Execution of this transition:
after that #2939 will add new functionality to introspect queues and change task priorities |
I've extracted that last piece into as separate ticket and will close this one #6950 |
The queue uses tc-lib-azqueue for its core "queueing" operations. This supports both queues for tasks for workers, and delay queues for things like deadlines and task claims.
Because it mixes so many things in there, this queue is not easily introspected to find out, say, what tasks are pending for a particular task queue.
Let's find a way to do task queueing that's linked to the tasks table in a way that allows querying (#2939) the tasks in the queue. The
pendingTasks
API method should be upgraded in the process to take account of cancelled tasks (https://bugzilla.mozilla.org/show_bug.cgi?id=1434851).Requires #2932.
The text was updated successfully, but these errors were encountered: