Skip to content

docs: Describe long evaluation and domain scoping in the feature matrix - #67

Open
kinyoklion wants to merge 2 commits into
mainfrom
devin/1790174874-java-matrix-refresh
Open

kinyoklion wants to merge 2 commits into
mainfrom
devin/1790174874-java-matrix-refresh

Conversation

@kinyoklion

@kinyoklion kinyoklion commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Brings the feature matrix up to date with capabilities the OpenFeature Java SDK added since the matrix was written (getLongEvaluation and domain-bound provider initialization in 1.22.0).

  • Providers row now states that long evaluation comes from the SDK default, which evaluates the flag as a double and rejects out-of-range or fractional values
  • Domains row now states the provider is not domain-scoped, so one instance can serve several domains

Requirements

  • I have added test coverage for new or changed functionality (documentation only)
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

None.

Implementation details

Verified against the OpenFeature Java SDK 1.22.1 FeatureProvider interface: getLongEvaluation has a default implementation delegating to getDoubleEvaluation with safe-range checks, and initialize(EvaluationContext, String domain) plus isDomainScoped() are defaults the provider correctly leaves alone — the LaunchDarkly client holds no per-domain state.

No behavior change was needed: overriding getLongEvaluation would not improve precision, because LaunchDarkly numeric flag values are JSON numbers surfaced as doubles.

Documentation only; no code or tests changed.

Link to Devin session: https://app.devin.ai/sessions/a47abf28ecd44130917b9b5131287221
Open in Devin Desktop: https://app.devin.ai/desktop/session/a47abf28ecd44130917b9b5131287221?variant=devin
Requested by: @kinyoklion


Note

Overview
Updates the README feature matrix to document OpenFeature Java SDK 1.22.x behavior—no runtime or test changes.

The Providers row now explains that long flag evaluation relies on the SDK default (getDoubleEvaluation with range/fraction checks), not a custom provider override. The Domains row clarifies that the provider is not isDomainScoped and a single instance can be registered for multiple domains because it holds no per-domain state.

Reviewed by Cursor Bugbot for commit b9c9413. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration

Copy link
Copy Markdown
Contributor

@cursor review

Comment thread README.md Outdated
Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
@kinyoklion
kinyoklion marked this pull request as ready for review September 23, 2026 16:22
@kinyoklion
kinyoklion requested a review from a team as a code owner September 23, 2026 16:22

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant