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

Support runtime cache of feature change timestamps #1035

Open
cportele opened this issue Aug 9, 2023 · 0 comments
Open

Support runtime cache of feature change timestamps #1035

cportele opened this issue Aug 9, 2023 · 0 comments

Comments

@cportele
Copy link
Member

cportele commented Aug 9, 2023

This is a follow-up to #994, which added support for the Last-Modified header for requests to a single feature. The timestamp can either be a fixed value in the configuration or taken from a feature property.

An alternative approach that would support concurrent editing via the CRUD operations (but not for cases, where the data is updated through other means) would be to

  • use the startup time of the ldproxy instance as the initial value for all features;
  • maintain a runtime registry with feature update timestamps that have been edited (empty at startup);
  • to determine the last-modified value for a feature, the registry would be checked and the startup time would be used as fallback;
  • potentially clean-up entries in the registry after some time (24 hours?).

If there are multiple instances of ldproxy in a cluster, the runtime registry would need to be shared across all containers and the startup time would need to be the startup time of the cluster.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant