Skip to content

Increase async cache stale period from 3 to 5 minutes.#608

Merged
renaudhartert-db merged 1 commit into
mainfrom
increase-async-cache-stale-period
Jan 10, 2026
Merged

Increase async cache stale period from 3 to 5 minutes.#608
renaudhartert-db merged 1 commit into
mainfrom
increase-async-cache-stale-period

Conversation

@renaudhartert-db

@renaudhartert-db renaudhartert-db commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Changes

Increases the default async cache stale period from 3 minutes to 5 minutes.

Rationale

The 5-minute stale period covers the maximum monthly downtime allowed by a 99.99% uptime SLA (~4.38 minutes). This ensures that even during the worst-case downtime scenario, the cache can continue serving stale tokens while waiting for the auth service to recover (assuming continuous traffic).

The 5-minute stale period covers the maximum monthly downtime allowed by a 99.99% uptime SLA (~4.38 minutes). This ensures that even during the worst-case downtime scenario, the cache can continue serving stale tokens while waiting for the auth service to recover (assuming continuous traffic).

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-java

Inputs:

  • PR number: 608
  • Commit SHA: e365fb4523d0cadd05158b3304bb45fd6948a09b

Checks will be approved automatically on success.

@renaudhartert-db renaudhartert-db requested review from parthban-db and tejaskochar-db and removed request for parthban-db January 9, 2026 21:43
@renaudhartert-db renaudhartert-db added this pull request to the merge queue Jan 10, 2026
Merged via the queue into main with commit 7e3eab2 Jan 10, 2026
16 checks passed
@renaudhartert-db renaudhartert-db deleted the increase-async-cache-stale-period branch January 10, 2026 14:15
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