Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions changes/4133.doc.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fixed the documented default of ``max_age_seconds`` in the ``CacheStore`` docstring: the default is ``"infinity"`` (no expiration), not ``None``, which is rejected. Also noted that ``cache_store`` must support deletes.
9 changes: 5 additions & 4 deletions src/zarr/experimental/cache_store.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,11 @@ class CacheStore(WrapperStore[Store]):
store : Store
The underlying store to wrap with caching
cache_store : Store
The store to use for caching (can be any Store implementation)
max_age_seconds : int | None, optional
Maximum age of cached entries in seconds. None means no expiration.
Default is None.
The store to use for caching (can be any Store implementation that
supports deletes)
max_age_seconds : int or "infinity", optional
Maximum age of cached entries in seconds. The string "infinity" means
entries never expire. Default is "infinity".
max_size : int | None, optional
Maximum size of the cache in bytes. When exceeded, least recently used
items are evicted. None means unlimited size. Default is None.
Expand Down
Loading