-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
The base client currently has a default expiry time for each entry, as well as a number of maximum items it can hold at once (I think this is first in first out).
If a Bluesky plan makes multiple requests using this client, and the second of these requests happens to be after the cache expiry time, or the cache fill up and drops the first entry, then the settings could change mid-plan without the user meaning to do this.
This seems like it could trip people up, but on the other hand, the default timeout if 1 hr - if we encourage people to make requests with reset_cache=True at the start of their plans, then this should be fine.
Acceptance Criteria
- Decide if default cache behaviour is good
- Document that users should use
reset_cache=Trueat the start of their plans, and guidance on theircache_sizecache_lifetime_svalues
Metadata
Metadata
Assignees
Labels
No labels