Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 修复事件绑定不稳定引用引发的内存泄漏问题 #2350

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zyy9803
Copy link

@zyy9803 zyy9803 commented Nov 23, 2024

修复由于事件绑定不稳定引用引发的内存泄漏问题

@zpqsunny
Copy link

zpqsunny commented Jan 9, 2025

你好,能否详细描述下内存泄漏后的表现形式吗?是页面崩溃吗? @zyy9803

@zyy9803
Copy link
Author

zyy9803 commented Jan 9, 2025

你好,能否详细描述下内存泄漏后的表现形式吗?是页面崩溃吗? @zyy9803

是的,我们在使用的过程中向 windowKeyEventHandler 事件传入了 widget 的引用,结果发现对于大看板(含有 80+ widget)每次编辑/查看后都会造成 30-80Mb 的内存上涨,且无法被 gc,分析师长时间编辑保存看板后,整个网页会因内存占用过大而崩溃。

@zpqsunny
Copy link

zpqsunny commented Jan 9, 2025

你好,能否详细描述下内存泄漏后的表现形式吗?是页面崩溃吗? @zyy9803

是的,我们在使用的过程中向 windowKeyEventHandler 事件传入了 widget 的引用,结果发现对于大看板(含有 80+ widget)每次编辑/查看后都会造成 30-80Mb 的内存上涨,且无法被 gc,分析师长时间编辑保存看板后,整个网页会因内存占用过大而崩溃。

非常感谢,有很多次遇到这种情况,但是一直找不到原因,我还想问一下,修改后奔溃的原因是彻底没有了呢?还是奔溃的次数相对减少很多?

@zyy9803
Copy link
Author

zyy9803 commented Jan 9, 2025

你好,能否详细描述下内存泄漏后的表现形式吗?是页面崩溃吗? @zyy9803

是的,我们在使用的过程中向 windowKeyEventHandler 事件传入了 widget 的引用,结果发现对于大看板(含有 80+ widget)每次编辑/查看后都会造成 30-80Mb 的内存上涨,且无法被 gc,分析师长时间编辑保存看板后,整个网页会因内存占用过大而崩溃。

非常感谢,有很多次遇到这种情况,但是一直找不到原因,我还想问一下,修改后奔溃的原因是彻底没有了呢?还是奔溃的次数相对减少很多?

彻底解决了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants