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

Remove superfluous database calls in deliver_email and send_email #477

Open
2 tasks
jimleroyer opened this issue Nov 29, 2024 · 0 comments
Open
2 tasks
Labels
Throughput Will boost our throughput

Comments

@jimleroyer
Copy link
Member

Description

As a user of GCNotify,
I want my emails to be sent quick,
So that I can have a super service sending emails real quick.

As a developer of GCNotify,
I want to clean up house around the performance of network and database calls in the deliver_email and send_email functions,
So that I can get faster performance for the email delivery.

WHY are we building?

Optimize speed on our sending delivery code path.

WHAT are we building?

We can speed up the deliver_email code before the send_email call by removing superfluous database calls. There are related caching performance improvements as well that this function would benefit from.

GDS did improvements that show what can be done. These improvements are described in more details in this task.

This suggestion comes from our ADR on Scalability: increasing send rates.

VALUE created by our solution

Email delivery speed.

Acceptance Criteria

  • Sending emails is faster.

QA Steps

  • Run the rollercoaster test before and after the change to compare the time differences.
@jimleroyer jimleroyer added the Throughput Will boost our throughput label Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Throughput Will boost our throughput
Projects
None yet
Development

No branches or pull requests

1 participant