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

feat(orca-sdk): implement cache invalidation for lookup table fetcher #98

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Jovian-Dsouza
Copy link

Add time-based cache invalidation to prevent stale data in OrcaLookupTableFetcher:

  • Add DEFAULT_CACHE_TTL (5 minutes) and configurable cacheTTL
  • Introduce CacheEntry type to track cache timestamps
  • Update cache structures to include invalidation logic
  • Add cache validation checks in lookup methods
  • Convert Set-based cache miss tracking to timestamp-based approach

Resolves TODO regarding cache invalidation path

@Jovian-Dsouza
Copy link
Author

@wjthieme could you please review this

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.

1 participant