Skip to content

Remove DAX from dev and stage#581

Merged
bheesham merged 6 commits intomozilla-iam:masterfrom
bheesham:add-dax-stage
Jul 30, 2025
Merged

Remove DAX from dev and stage#581
bheesham merged 6 commits intomozilla-iam:masterfrom
bheesham:add-dax-stage

Conversation

@bheesham
Copy link
Contributor

Small writeup: 2025-07-25 DynamoDB Accelerator does not improve response times for Person API

We found that adding DAX does not improve response times after trying a couple of different things. The writeup has more deets, but the gist of it is:

  • we can see profile retrieval making use of it;
  • we can see we're hitting the cache, and not going to Dynamo;
  • we can see response times worsen.

Same behaviour when trying with different replicas.

I'm including the code here for posterity, should we want to revive this at some point. Plus, there were a couple of bug fixes along the way which seemed like a good idea to include.

Jira: IAM-1732

bheesham added 4 commits July 30, 2025 11:18
There's some resources which are unique per AWS account, such as policies and
roles. Here I use the `name_prefix` option of these resources to create unique
versions of them.

Jira: IAM-1732
The testing environment has _way_ more data than dev, and should provide a
better environment for running tests.

Jira: IAM-1732
Jira: IAM-1732
Copy link
Member

@bkochendorfer bkochendorfer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for cleaning up the experiment

@bheesham bheesham changed the title Remove DAX from test and stage Remove DAX from dev and stage Jul 30, 2025
bheesham added 2 commits July 30, 2025 11:48
Since DAX offers no benefits, as found on dev and test, we won't be using it in
production.

Unfortunate, but this closes off a thread of work and allows us to focus on
other things. No "easy" wins here.

Jira: IAM-1732
@bheesham bheesham merged commit 6acff60 into mozilla-iam:master Jul 30, 2025
1 check passed
@bheesham bheesham deleted the add-dax-stage branch July 30, 2025 15:55
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

Successfully merging this pull request may close these issues.

2 participants