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

RemoteEvent.get Caching #40

Open
bjendres opened this issue Mar 16, 2023 · 1 comment
Open

RemoteEvent.get Caching #40

bjendres opened this issue Mar 16, 2023 · 1 comment
Labels
enhancement New feature or request feature new feature
Milestone

Comments

@bjendres
Copy link
Member

bjendres commented Mar 16, 2023

For event list queries (e.g. "all events that still have open sports") we should allow for short-term caching, provided the request doesn't contain a personalised token.

Currently there is no such caching in the API, the runtime might be high, and the results don't vary too much. Therefore a cache based on the complete request, with a short TTL (e.g. 10 seconds) could improve performance on highly frequented systems.

Of course, any changes to events and/or participants should invalidate the cache.

The fact that people spend 10's of seconds on the registration form alone, illustrates that an additional cache with short TTL shouldn't have too much impact on the overall user experience.

@bjendres bjendres added enhancement New feature or request feature new feature labels Mar 16, 2023
@jensschuppe jensschuppe added this to the 1.2 milestone Oct 17, 2023
@jensschuppe jensschuppe modified the milestones: 1.2, 1.3 Mar 25, 2024
@jensschuppe
Copy link
Contributor

Moving to 1.3 as 1.2 is already in beta.

@jensschuppe jensschuppe modified the milestones: 1.3, 1.4 Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature new feature
Projects
None yet
Development

No branches or pull requests

2 participants