Add readonly mode #428
Merged
Add readonly mode #428
+76
−13
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fix: #423 When you know that the cache won't be used again, it avoid some useless work and IOs. Typically this might be the case for dockerized applications. You generate a cache when building the image, but then when you boot the application any cache update won't be persisted, so thre is no point.
casperisfine
force-pushed
the
readonly-mode
branch
from
a011112 to
b51397f
Compare
|
Thanks again Jean, are you planning to cut a new release? |
|
I can cut a release easily yeah, but would be good if you double checked the use case you had works as expected. You can point the gemfile at the repo. |
technicalpickles
added a commit
to technicalpickles/rails
that referenced
this pull request
Nov 15, 2023
This was added in back in bootsnap 1.15.0 (Shopify/bootsnap#428). This can save extra work and IO when the cache won't be changed, like is the case for docker containers
4 tasks
technicalpickles
added a commit
to technicalpickles/rails
that referenced
this pull request
Nov 16, 2023
This was added in back in bootsnap 1.15.0 (Shopify/bootsnap#428). This can save extra work and IO when the cache won't be changed, like is the case for docker containers
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix: #423
When you know that the cache won't be used again, it avoid some useless work and IOs.
Typically this might be the case for dockerized applications. You generate a cache when building the image, but then when you boot the application any cache update won't be persisted, so thre is no point.