Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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