-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
bugfix: fix direct buffer OOM #2175
Conversation
Signed-off-by: slievrly <slievrly@163.com>
Codecov Report
@@ Coverage Diff @@
## develop #2175 +/- ##
=============================================
- Coverage 53.16% 53.14% -0.02%
+ Complexity 2515 2513 -2
=============================================
Files 485 485
Lines 15346 15348 +2
Branches 1770 1770
=============================================
- Hits 8158 8156 -2
- Misses 6403 6406 +3
- Partials 785 786 +1
|
This can be useful by using the newest snapshot. |
已验证,oom问题已解决 |
3 similar comments
已验证,oom问题已解决 |
已验证,oom问题已解决 |
已验证,oom问题已解决 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
true, | ||
WORKER_THREAD_SIZE, | ||
WORKER_THREAD_SIZE, | ||
2048 * 64, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[2048 * 64] Is this property set too large, 16 times the default value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Do not merge for now, collect more feedback. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: slievrly slievrly@163.com
Ⅰ. Describe what this PR did
bugfix: fix direct buffer OOM
Ⅱ. Does this pull request fix one issue?
fix #2164
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews