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 N+1 queries in /api/v1/resource/ #1336

Open
Tracked by #1333
sainak opened this issue Jun 2, 2023 · 4 comments
Open
Tracked by #1333

Fix N+1 queries in /api/v1/resource/ #1336

sainak opened this issue Jun 2, 2023 · 4 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@sainak
Copy link
Member

sainak commented Jun 2, 2023

Sentry Issue: CARE-M5

Transaction Name /api/v1/resource/
Parent Span django.view - resource-request-list
Repeating Spans (0) db - SELECT COUNT(*) AS __count FROM facility_patientregistration WHERE (facility_patientregistration.deleted = %s AND facility_patientregistration.facility_id = %s AND facility_patientregistration.is_active = %s)
db - SELECT COUNT(*) AS __count FROM facility_bed WHERE (facility_bed.deleted = %s AND facility_bed.facility_id = %s)
@sainak sainak mentioned this issue Jun 2, 2023
7 tasks
@sainak sainak added the good first issue Good for newcomers label Jun 6, 2023
@adityasuthar
Copy link

Hi @sainak can I work on this issue

@adityasuthar
Copy link

Parent Span | django.view - resource-request-list

I could not find this Django view in the current version of develop branch. It has been some time since this issue was raised and I think the view class name changed in the meantime. Can you help me finding the view and the query which we need to optimise @sainak @rithviknishad

@adityasuthar
Copy link

I found the view ResourceRequestViewSet, working on it.

@adityasuthar
Copy link

Hi @sainak @rithviknishad I have raised a pull request at #2035

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants