diff options
author | Sean <squahtx@users.noreply.github.com> | 2021-09-01 11:55:31 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-09-01 11:55:31 +0100 |
commit | f8bf83b811cf54b4ce668bcda6043d14c4980f00 (patch) | |
tree | 5b0bc0a4bbbf8011f5cc7f02ac79ad155d13f33e /synapse | |
parent | Allow configuration of the oEmbed URLs. (#10714) (diff) | |
download | synapse-f8bf83b811cf54b4ce668bcda6043d14c4980f00.tar.xz |
Skip the final GC on shutdown to improve restart times (#10712)
Use `gc.freeze()` on exit to exclude all existing objects from the final GC. In testing, this sped up shutdown by up to a few seconds. `gc.freeze()` runs in constant time, so there is little chance of performance regression. Signed-off-by: Sean Quah <seanq@element.io>
Diffstat (limited to 'synapse')
-rw-r--r-- | synapse/app/_base.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/synapse/app/_base.py b/synapse/app/_base.py index 39e28aff9f..6fc14930d1 100644 --- a/synapse/app/_base.py +++ b/synapse/app/_base.py @@ -12,6 +12,7 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. +import atexit import gc import logging import os @@ -403,6 +404,12 @@ async def start(hs: "HomeServer"): gc.collect() gc.freeze() + # Speed up shutdowns by freezing all allocated objects. This moves everything + # into the permanent generation and excludes them from the final GC. + # Unfortunately only works on Python 3.7 + if platform.python_implementation() == "CPython" and sys.version_info >= (3, 7): + atexit.register(gc.freeze) + def setup_sentry(hs): """Enable sentry integration, if enabled in configuration |