From f8431b3a19c9e15f709b5c686c6adbeadb222b04 Mon Sep 17 00:00:00 2001 From: Wenzhao Hu <12122021+wzhudev@users.noreply.github.com> Date: Wed, 5 Feb 2025 12:51:57 +0800 Subject: [PATCH] fix: memory leak temp solution --- packages/engine-render/src/render-manager/render-unit.ts | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/packages/engine-render/src/render-manager/render-unit.ts b/packages/engine-render/src/render-manager/render-unit.ts index 8a037145b33..31d9c6e1b27 100644 --- a/packages/engine-render/src/render-manager/render-unit.ts +++ b/packages/engine-render/src/render-manager/render-unit.ts @@ -128,6 +128,15 @@ export class RenderUnit extends Disposable implements IRender { this._activated$.next(false); this._activated$.complete(); + + // Avoid memory leak. Basically it is because RenderUnit itself is leaking. + // We use this as a temporary solution to make CI pass. + // @ts-ignore + this._renderContext.activated$ = null; + // @ts-ignore + this._renderContext.activate = null; + // @ts-ignore + this._renderContext.deactivate = null; } /**