-
Notifications
You must be signed in to change notification settings - Fork 45
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
aepfli
added
good first issue
Good for newcomers
help wanted
Extra attention is needed
labels
Jan 16, 2025
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]>
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: