-
Notifications
You must be signed in to change notification settings - Fork 49
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
[flagd] improve wait logic to a more elegant solution #1160
Labels
Comments
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 20, 2025
Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 21, 2025
…ant-solution-open-feature#1160 # Conflicts: # providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/FlagdProvider.java # providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/resolver/process/InProcessResolver.java # providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/InProcessResolverTest.java
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 21, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 21, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 21, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 21, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 22, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 22, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Jan 22, 2025
…re#1160 Signed-off-by: christian.lutnik <[email protected]>
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Feb 17, 2025
…open-feature#1160 # Conflicts: # providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/FlagdProvider.java
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Feb 19, 2025
…open-feature#1160 # Conflicts: # providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/FlagdProvider.java # providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/EventSteps.java
toddbaert
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Feb 26, 2025
chrfwow
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Feb 26, 2025
…gant-solution-#1160' into improve-wait-logic-to-a-more-elegant-solution-open-feature#1160
toddbaert
added a commit
to chrfwow/java-sdk-contrib
that referenced
this issue
Feb 26, 2025
toddbaert
added a commit
that referenced
this issue
Feb 26, 2025
Signed-off-by: christian.lutnik <[email protected]> Co-authored-by: Todd Baert <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Our current waiting logic could be improved. It is a working wait logic, but it might not be the most elegant solution.
java-sdk-contrib/providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/resolver/common/Util.java
Lines 26 to 38 in cd0ea9e
This task aims to migrate to an approach with
notify()
andwait()
. As suggested by @chrfwow within #1115.The text was updated successfully, but these errors were encountered: