Skip to content

feat: add the ability to control the cache#8311

Merged
mscolnick merged 1 commit intomarimo-team:mainfrom
aqora-io:feat-add-app-session-cache
Mar 12, 2026
Merged

feat: add the ability to control the cache#8311
mscolnick merged 1 commit intomarimo-team:mainfrom
aqora-io:feat-add-app-session-cache

Conversation

@jpopesculian
Copy link
Contributor

Reimplements the feature reverted by #8281
Goes best with #8310 to avoid invalid session caches

Add a new serve_cached_sessions_in_apps runtime configuration item
to use the cache in a read-only manner when running as an application

@vercel
Copy link

vercel bot commented Feb 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Mar 2, 2026 9:22am

Request Review

@dmadisetti
Copy link
Collaborator

Happy with this once #8310 passes. If you don't mind grabbing the pre-commit failure that would be great!

and use a read-only cache for run mode
@jpopesculian
Copy link
Contributor Author

@dmadisetti my bad! my vim configuration was handling ruff weird. should be fixed. don't know about the rest of the tests

@mscolnick mscolnick requested a review from dmadisetti March 2, 2026 14:54
@stubbi
Copy link

stubbi commented Mar 9, 2026

@dmadisetti is it good to go and be merged?

@dmadisetti
Copy link
Collaborator

@stubbi thanks for the ping. Merged in #8310

Just manually tested and continually got cache misses- but looking into it right now

@mscolnick mscolnick merged commit b75c6ec into marimo-team:main Mar 12, 2026
35 of 44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bash-focus Area to focus on during release bug bash enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants