Skip to content
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.

Sign up for GitHub

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

Jump to bottom

RQ: Missing currently executing queries view #4558

Merged
merged 5 commits into from
Feb 3, 2020
Merged

RQ: Missing currently executing queries view #4558

merged 5 commits into from
Feb 3, 2020

Conversation

rauchy
Copy link
Contributor

@rauchy rauchy commented Feb 2, 2020

With Celery we used to have a tab that showed currently running queries. We need to port it to RQ (currently it’s missing).

@arikfr arikfr added the Backend label Jan 19, 2020
@arikfr arikfr added this to To do in Switch from Celery to RQ via automation Jan 19, 2020
@weekly-digest weekly-digest bot mentioned this pull request Jan 20, 2020
Closed
@rauchy rauchy moved this from To do to In progress in Switch from Celery to RQ Feb 2, 2020
@rauchy
Copy link
Contributor

rauchy commented Feb 2, 2020

image

Anything missing?

  • I've added default sorting to the queues table (this bothered me for a long time)
  • Also I've added pagination to all tables

@rauchy rauchy requested a review from arikfr February 2, 2020 12:35
@rauchy rauchy moved this from In progress to Review in progress in Switch from Celery to RQ Feb 3, 2020
@weekly-digest weekly-digest bot mentioned this pull request Feb 3, 2020
Closed
arikfr
arikfr reviewed Feb 3, 2020
View reviewed changes
Copy link
Member

@arikfr arikfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@rauchy rauchy merged commit 7a34a76 into master Feb 3, 2020
Switch from Celery to RQ automation moved this from Review in progress to Done Feb 3, 2020
@rauchy rauchy deleted the show-current-query-executions branch February 3, 2020 21:51
@weekly-digest weekly-digest bot mentioned this pull request Feb 10, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arikfr arikfr arikfr left review comments

Assignees

@rauchy rauchy

Labels
Backend
Projects
No open projects
Switch from Celery to RQ
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rauchy @arikfr