[5.5] Allow for configuration of sticky databases #20746
Merged
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.
Allow #20445 to be configurable
I can forsee edge cases where people might not want this functionality (i.e. doing a small write followed by unrelated large read in same request - such as an application that logs all user actions) - so we can easily make it configurable for those that want either behavior.
Also this way we preserve current behavior - because the config will be
null
on existing applications (since the value doesnt exist) - so this reduces risk of strange breaking behavior during a 5.5 upgrade.Instead I propose to mention this new config option in the 5.5 upgrade guide, for those that want to enable it. And meanwhile I can do another PR for laravel/laravel to include it in the default config stack and in the docs.
I'll do the other PR's for you if this gets accepted.