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
[2.7] fix memory leak #10231
[2.7] fix memory leak #10231
Conversation
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.
flacoco flags 5 suspicious lines as the potential root cause of the test failure.
dubbo-common/src/main/java/org/apache/dubbo/common/io/UnsafeByteArrayOutputStream.java
Outdated
Show resolved
Hide resolved
dubbo-common/src/main/java/org/apache/dubbo/common/io/UnsafeByteArrayOutputStream.java
Outdated
Show resolved
Hide resolved
dubbo-common/src/main/java/org/apache/dubbo/common/io/UnsafeByteArrayOutputStream.java
Outdated
Show resolved
Hide resolved
dubbo-common/src/main/java/org/apache/dubbo/common/io/UnsafeByteArrayInputStream.java
Outdated
Show resolved
Hide resolved
dubbo-common/src/main/java/org/apache/dubbo/common/io/UnsafeByteArrayInputStream.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #10231 +/- ##
============================================
+ Coverage 60.95% 60.98% +0.02%
- Complexity 448 449 +1
============================================
Files 1100 1100
Lines 44579 44511 -68
Branches 6493 6486 -7
============================================
- Hits 27175 27143 -32
+ Misses 14437 14404 -33
+ Partials 2967 2964 -3
Continue to review full report at Codecov.
|
What is the purpose of the change
fix #10219
Brief changelog
remove
ThreadLocal
fromHessian2ObjectInput
andHessian2ObjectOutput