-
Notifications
You must be signed in to change notification settings - Fork 26.4k
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
feat: fix #13327 #13334
feat: fix #13327 #13334
Conversation
- dubbo.jstack-dump.max-line can be used to indicate that all stack trace lines should be printed now - write stack trace info directly into OutputStream instead of building a String
Codecov Report
@@ Coverage Diff @@
## 3.2 #13334 +/- ##
============================================
+ Coverage 65.44% 69.54% +4.10%
Complexity 2 2
============================================
Files 2006 1652 -354
Lines 86319 71608 -14711
Branches 12755 10265 -2490
============================================
- Hits 56494 49802 -6692
+ Misses 24497 17111 -7386
+ Partials 5328 4695 -633 see 512 files with indirect coverage changes 📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today! |
dubbo-common/src/main/java/org/apache/dubbo/common/utils/JVMUtil.java
Outdated
Show resolved
Hide resolved
Signed-off-by: crazyhzm <crazyhzm@gmail.com>
# Conflicts: # dubbo-dependencies-bom/pom.xml # dubbo-dependencies/dubbo-dependencies-zookeeper-curator5/pom.xml # dubbo-dependencies/dubbo-dependencies-zookeeper/pom.xml # pom.xml
Bumps `netty4_version` from 4.1.100.Final to 4.1.101.Final. Updates `io.netty:netty-all` from 4.1.100.Final to 4.1.101.Final - [Commits](netty/netty@netty-4.1.100.Final...netty-4.1.101.Final) Updates `io.netty:netty-bom` from 4.1.100.Final to 4.1.101.Final - [Commits](netty/netty@netty-4.1.100.Final...netty-4.1.101.Final) --- updated-dependencies: - dependency-name: io.netty:netty-all dependency-type: direct:development update-type: version-update:semver-patch - dependency-name: io.netty:netty-bom dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
fileOutputStream.set(jStackStream); | ||
JVMUtil.jstack(jStackStream); |
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.
Please add some test cases to verify the result
- dubbo.jstack-dump.max-line can be used to indicate that all stack trace lines should be printed now - write stack trace info directly into OutputStream instead of building a String
…into opt-jvm-prt-stk
Due to a branch operation error, this PR is reopened in #13358. |
Kudos, SonarCloud Quality Gate passed! |
What is the purpose of the change
fix #13327
dubbo.jstack-dump.max-line
can be used to indicate that all stack trace lines should be printed now by setting it to a negative number, -1 for example.Brief changelog
update org.apache.dubbo.common.utils.JVMUtil
Verifying this change
Checklist