-
Notifications
You must be signed in to change notification settings - Fork 12
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
[BUG] 内存泄漏? #6
Comments
是remix版本吗? |
是 douyuex.com 的版本 |
我对代码初步进行了分析,觉得没有特别离谱的内存泄露处(有隐患但是是业务需要,而且也不至于造成out of memory) 你是否将 说一下我做了什么吧: |
好的 我会进行排查
…---原始邮件---
发件人: ***@***.***>
发送时间: 2023年4月20日(周四) 凌晨2:17
收件人: ***@***.***>;
抄送: ***@***.******@***.***>;
主题: Re: [qianjiachun/douyu-monitor] [BUG] 内存泄漏? (Issue #6)
这是另一个会导致内存泄漏的BUG
在互联网网络中断的时候,它似乎会在不断的重连啥的,但似乎没有释放失败的重试,导致进程CPU和内存占用越来越高。
大概是重试逻辑的问题,可能是设计时没考虑到长时间断网还会挂着页面之类的。
(这个是在我断网的时候突然发现内存不足时看看是谁导致的发现这个进程内存爆炸的,平时怎么挂都没有什么问题,所以我感觉应该是这个原因)
编程我只懂皮毛,平时也就顺手做一下QA测试,代码上谈不上有能力review,没办法从你大量的代码里找出相关的部分。
你如果不嫌弃花时间跟一个外行讨论的话可以把相关的代码贴出来解说一下,我也许可以从程序设计的方向上分析一下。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
用浏览器长时间(比如全天挂机)运行后页面会崩溃,错误代码为:Out of memory
测试浏览器为最新 Stable 版本的 Chrome 和 Edge。Edge 需关闭效率模式。
The text was updated successfully, but these errors were encountered: