Skip to content

add pending property to QueueInfo#148

Merged
gresrun merged 1 commit into
gresrun:masterfrom
uberall:pending-jobs
Aug 26, 2018
Merged

add pending property to QueueInfo#148
gresrun merged 1 commit into
gresrun:masterfrom
uberall:pending-jobs

Conversation

@bp-FLN

@bp-FLN bp-FLN commented Aug 20, 2018

Copy link
Copy Markdown
Contributor

For delayed queues, the size property may be misleading,
as it shows all jobs that are enqueued in that queue.

This patch adds a pending property to the QueueInfo class
which shows the number of jobs in a delayed queue that are
eligible for execution by a worker.

For delayed queues, the size property may be misleading,
as it shows all jobs that are enqueued in that queue.

This patch adds a `pending` property to the QueueInfo class
which shows the number of jobs in a delayed queue that are
eligible for execution by a worker.
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.08%) to 70.774% when pulling 76cc6db on uberall:pending-jobs into c0e56a1 on gresrun:master.

@gresrun
gresrun merged commit 7e40392 into gresrun:master Aug 26, 2018
@bp-FLN
bp-FLN deleted the pending-jobs branch August 27, 2018 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants