-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-28242: Adapts ProfileServlet for async-profiler 2.x #5566
base: master
Are you sure you want to change the base?
Conversation
lfrancke
commented
Dec 6, 2023
- Adds settings that are available in async-profiler 2.0 and later and removes those that don't exist anymore.
- Adds support for the itimer event, renames "branches" to "branch-instructions"
- Removes support for SVG and SUMMARY output and makes FLAMEGRAPH the default
- Adds settings that are available in async-profiler 2.0 and later and removes those that don't exist anymore. - Adds support for the itimer event, renames "branches" to "branch-instructions" - Removes support for SVG and SUMMARY output and makes FLAMEGRAPH the default
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
The test failures seem unrelated. |
The general check is mostly for testing compiling and install. They other two checks are for running tests against jdk8 and jdk11. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Yeah makes sense, even 3.0 is released. the last 1.x release was 2+ years ago. |
Yeah, I did start a thread on the mailing list and need to get around to update this PR. |