Skip to content

Feature: running queries monitor #856

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
Merged
merged 1 commit into from
Apr 18, 2016
Merged

Feature: running queries monitor #856

merged 1 commit into from
Apr 18, 2016

Conversation

arikfr
Copy link
Member

@arikfr arikfr commented Feb 25, 2016 •
edited
Loading

First iteration of adding views that show currently executing queries, queries that waiting in queue and ones that recently finished. Also showing the "outdated queries" list (queries that are scheduled to refresh).

TODO:

  • Add periodic job to remove old finished tasks.
  • Cancel buttons.

re_dash___running_queries

@arikfr arikfr mentioned this pull request Mar 8, 2016
Open
@ghost
Copy link

ghost commented Mar 10, 2016

+1 +1 +1!!!!

@arikfr arikfr force-pushed the feature/task_monitoring branch 2 times, most recently from 60f42e4 to b1da7a9 Compare March 23, 2016 09:44
@arikfr
Copy link
Member Author

arikfr commented Mar 23, 2016

There is still an issue here, that we inherit from Celery: if the worker are forcefully killed (kill -9) we won't detect this. I'm not sure how to handle this, aside from:

  1. Adding some explanation around this in documentation.
  2. Marking jobs as cancelled after 24 hours, assuming that no one is running such long queries.

@arikfr arikfr force-pushed the feature/task_monitoring branch from 85d7696 to f624fe6 Compare April 17, 2016 18:59
@arikfr arikfr changed the title WIP Feature: running queries monitor Feature: running queries monitor Apr 18, 2016
@arikfr
Feature: running queries (tasks) monitor
d38ab20 
- Refactored tasks module into a package.
- Add new admins screens (running queries & outdated queries).
@arikfr arikfr force-pushed the feature/task_monitoring branch from 3e70660 to d38ab20 Compare April 18, 2016 10:47
@arikfr arikfr merged commit 30227f4 into master Apr 18, 2016
@arikfr arikfr deleted the feature/task_monitoring branch July 16, 2016 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@arikfr