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.

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

Fix the "Current Activity" link. #224

Closed
kfogel opened this issue Feb 28, 2020 · 3 comments
Closed

Fix the "Current Activity" link. #224

kfogel opened this issue Feb 28, 2020 · 3 comments

Comments

@kfogel
Copy link
Member

kfogel commented Feb 28, 2020

Wish I had the full URL to that link -- @OhMcGoo has it, though. I think it's different from "Currently Open Incidents" (incidents/CHAPTER_SHORTNAME/incident/activity).

There's such a long wait time right now that apparently even Jim has never seen the inside of this chamber in the temple.

@frankduncan
Copy link
Contributor

Did some research. When running in development, this returns (using a copy of a production database). Not extremely fast, but it is returning a page. There doesn't look to be anything too egregious in the sql query. Then, when updating the log level in production so I can see what queries are happening and which one is taking so long, it started returning results there too.

This needs more looking into, but it's looking less like a software problem and more like a server problem.

@frankduncan
Copy link
Contributor

Adding an index did improve the slowest query in development by a factor of 1000, so that hopefully that will be enough to make production snappy. Will need to take a look when this fix makes its way there.

frankduncan pushed a commit that referenced this issue Jun 24, 2020
The Current Activity link was slow and in production the versions
lookup was taking 5-40 seconds.  In development, it was taking 1.5
seconds, and this index improved that to 1.5 milliseconds.

Issue #224: Fix the "Current Activity" link.
frankduncan added a commit that referenced this issue Jun 24, 2020
@jim-mcgowan
Copy link
Member

OK, thanks @frankduncan. I'm still not quite sure how the Current Activity feature will get used operationally but I sense it will be really useful for DAT Managers who want to see what's going on without having to open individual incident reports.

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

No branches or pull requests

3 participants