Skip to content
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

feat: add note cache is no longer experimental #547

Closed
wants to merge 5 commits into from

Conversation

adriantam
Copy link
Member

Description

Cache is no longer considered experimental

References

https://github.com/openfga/openfga/releases/tag/v1.3.6

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@adriantam adriantam requested a review from a team as a code owner November 7, 2023 00:54
@adriantam adriantam requested a review from jon-whit November 7, 2023 01:09
|-------------------|--------------------------------------------------------------------|-----------------------|-------------------------------|
| otel-metrics | Enables support for exposing OpenFGA metrics through OpenTelemetry | `0.3.2 <= v < v0.3.5` | became default in `v0.3.5` |
| list-objects | Enables ListObjects API | `0.2.0 <= v < v0.3.3` | became default in `v0.3.3` |
| check-query-cache | Enables caching of check subproblem result | `1.3.1 <= v < v1.3.6` | became supported from `v1.3.6`|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here and above, I'm not sure what this is supposed to mean. I wonder if we can drop the "Status" column in favor of just the explicit version range?

An experimental flag is present in a release until we remove it. Once we remove it I wouldn't say it is "defaulted", because it is is gone. It no longer is interpreted in any way by the server. So I think it is less confusing if we just say the range of OpenFGA server versions that the experimental flag existed in.

WDYT?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree - I have trouble coming up with what to write in the status section. What do you think @rhamzeh ?

@adriantam
Copy link
Member Author

Close in light of #594

@adriantam adriantam closed this Dec 18, 2023
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.

2 participants