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

Twilio Status Override Celery Task: SQL Performance Update #2105

Open
3 tasks
mchlwellman opened this issue Nov 5, 2024 · 0 comments
Open
3 tasks

Twilio Status Override Celery Task: SQL Performance Update #2105

mchlwellman opened this issue Nov 5, 2024 · 0 comments

Comments

@mchlwellman
Copy link

User Story - Business Need

The Twilio status updating Celery job is querying the database for Notifications, and it is taking far too long to return results. See this example below where it took 12 seconds:

Image

This query needs to be revisited to be more performant, possibly by:

  • Removing the ordering

  • Adding an index on some of the where conditions, sent_at and notification_status

  • Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)

As a VANotify developer
I want backend tasks to run as efficiently as possible
So that resources have high availability

Additional Info and Resources

  • The SQL query is here

Acceptance Criteria

  • The Celery task database query executes in a reasonable amount of time (TBD)
  • This work is added to the sprint review slide deck (key win bullet point and demo slide)

QA Considerations

  • Should be no functional changes, just performance

Potential Dependencies

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants