Skip to content

fix: flagd caching #581

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

Merged

Conversation

Kavindu-Dodan
Copy link
Collaborator

This PR

  • Fix flagd cache handling : Caching never worked as enum.valueOf(String) always throwed and with no exception loggig we never understood this happening
  • Improves logic and adds testing

Signed-off-by: Kavindu Dodanduwa <[email protected]>
Signed-off-by: Kavindu Dodanduwa <[email protected]>
Signed-off-by: Kavindu Dodanduwa <[email protected]>
@Kavindu-Dodan Kavindu-Dodan force-pushed the fix/cache-type-resolving branch from d65783d to 0f75eef Compare December 11, 2023 16:57
…s/flagd/resolver/grpc/cache/Cache.java

Co-authored-by: Todd Baert <[email protected]>
Signed-off-by: Kavindu Dodanduwa <[email protected]>
@toddbaert toddbaert self-requested a review December 11, 2023 20:51
@Kavindu-Dodan Kavindu-Dodan merged commit e953fef into open-feature:main Dec 11, 2023
DBlanchard88 pushed a commit to DBlanchard88/java-sdk-contrib that referenced this pull request Apr 29, 2024
…ature#581)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

4 participants