-
Notifications
You must be signed in to change notification settings - Fork 69
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
Conversation
|-------------------|--------------------------------------------------------------------|-----------------------|-------------------------------| | ||
| 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`| |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 ?
Close in light of #594 |
Description
Cache is no longer considered experimental
References
https://github.com/openfga/openfga/releases/tag/v1.3.6
Review Checklist
main