Skip to content

Fix cache expiration-related issues in Auto Polling mode#98

Merged
adams85 merged 5 commits intomasterfrom
fix/autopolling-cache-expiration
Sep 6, 2024
Merged

Fix cache expiration-related issues in Auto Polling mode#98
adams85 merged 5 commits intomasterfrom
fix/autopolling-cache-expiration

Conversation

@adams85
Copy link
Contributor

@adams85 adams85 commented Sep 5, 2024

Describe the purpose of your pull request

This PR fixes multiple issues in Auto Polling mode:

  • Cache expiration should be checked in every poll iteration, not just in the first one to reduce network traffic when the SDK uses a shared cache.
  • Cache synchronization should happen even in offline mode in every poll iteration, not just in the first one to update in-memory cache when the SDK uses a shared cache.

Related issues (only if applicable)

https://trello.com/c/gnqqRydc

Requirement checklist (only if applicable)

  • I have covered the applied changes with automated tests.
  • I have executed the full automated test set against my changes.
  • I have validated my changes against all supported platform versions.
  • I have read and accepted the contribution agreement.

…a tolerance of 500ms to prevent missing fetches due to timer inaccuracy + sync with cache even in offline mode
@adams85 adams85 requested a review from a team as a code owner September 5, 2024 16:08
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 5, 2024

@adams85 adams85 merged commit 06408f5 into master Sep 6, 2024
@adams85 adams85 deleted the fix/autopolling-cache-expiration branch September 6, 2024 14:35
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.

3 participants