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

Improve Apdex score in contact details page #8994

Closed
latin-panda opened this issue Apr 9, 2024 · 1 comment · Fixed by #8995
Closed

Improve Apdex score in contact details page #8994

latin-panda opened this issue Apr 9, 2024 · 1 comment · Fixed by #8995
Assignees
Labels
Type: Performance Make something faster
Milestone

Comments

@latin-panda
Copy link
Contributor

latin-panda commented Apr 9, 2024

Describe the performance issue
Care Teams KR
Initial research done

I found out that load_reports (code) is requesting the reports' docs twice, the tech explanation is:

  • We use the search feature to return the reports (and docs) from the selected contact and its descendants (code)
  • The search feature uses the data record service (code) to get the docs. Makes a request to the user's local db
  • Then we take the ids from the search result and build the data for the report's header (code)
  • To build that header, we are retrieving the reports' docs again by using the data record service that also request the user's local db (but somewhere else in the code)

Instead of retrieving the same information twice, reports' docs, we can find a way to pass that information to the process that generates the reports' headers and void querying the user's local DB twice. And make sure we are not breaking anything else using the same processes.

@latin-panda latin-panda added the Type: Performance Make something faster label Apr 9, 2024
@latin-panda latin-panda added this to the 4.7.0 milestone Apr 9, 2024
@latin-panda latin-panda self-assigned this Apr 9, 2024
@latin-panda
Copy link
Contributor Author

FYI @ralfudx I'm working on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Performance Make something faster
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant