- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Long time ago, but there may be a memory impact/leak from this change on large sites 🐛 Memory usage increases linearly when (un)installing modules via config import Active
We currently try to rebuild the router in kernel.terminate so that it only gets rebuilt once per request. This is an optimization for when multiple modules (or other route rebuilding operations) are triggered in a single request.
However this makes module enabling non-transactional (see related issues) - the state of the installed modules and their routes gets out of sync, which can lead to race conditions and fatal errors.
Additionally, when enabling modules via the UI, it should move into a batch anyway (see related issues), so would get rebuilt every batch request anyway.
When enabling a module, just immediately rebuild routes instead of setting rebuildNeeded.
Fixed
8.1 ⚰️
Enhances developer experience.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Long time ago, but there may be a memory impact/leak from this change on large sites 🐛 Memory usage increases linearly when (un)installing modules via config import Active