Skip to content

Honor disabled cache regions after initialization - #266

Merged
amol- merged 1 commit into
masterfrom
amol/beaker-issue-215
Oct 27, 2026
Merged

amol- merged 1 commit into
masterfrom
amol/beaker-issue-215

Conversation

@amol-

@amol- amol- commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

Initialized cache-region decorators continue serving cached results after their region is disabled. Check the live enabled flag using the existing per-invocation region lookup, while retaining the initialized cache for reenabling. Other region options keep their existing behavior.

The production fix adds exactly two lines.

Validation:

  • Regression reproduced the stale cached result before the fix.
  • 24 decorator/cache-manager tests passed after the fix.
  • Coverage verifies disabled calls execute the original function with correctly bound arguments and reenabling reuses the prior cached value.
  • git diff --check passed; code-quality review approved the unchanged diff.

Fixes #215

@amol-
amol- merged commit eff9037 into master Oct 27, 2026
7 checks passed
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.

Incapable to disable cache after modified region settings

1 participant