Skip to content

Added section about async profiler with cf java #1771

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

Merged
merged 19 commits into from
May 6, 2025
Merged

Conversation

davidhunglam
Copy link
Contributor

Added section about how to use the https://github.com/SAP/cf-cli-java-plugin to run profiling with asprof.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling Mistakes

  • java/operating-applications/optimizing.md:56:8 Unknown word "Asprof"
  • java/operating-applications/optimizing.md:57:34 Unknown word "asprof"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Choosing an output filename with .html ending indicates to use the output type `flamegraph`, see [FlameGraph visualization](https://github.com/async-profiler/async-profiler/blob/master/docs/GettingStarted.md#flamegraph-visualization)
:::

`.html` files can be opened in a browser, `.jfr` files can be viewed with a viewer such as [OpenJDK Mission Control](https://openjdk.org/projects/jmc/).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can jfr files also be opened in intellij or vscode?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In IntelliJ only with the Ultimate Edition. For VSCode I'm not aware of any extension that could open jfr files.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Johannes hat ein PLugin für intellij gebaut - nur das ist eher noch instabil. Du kannst ihn ja fragen, ob wir es schon verlinken sollen

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting Errors

  • java/operating-applications/optimizing.md:73 Fenced code blocks should have a language specified
  • java/operating-applications/optimizing.md:78 Fenced code blocks should have a language specified
  • java/operating-applications/optimizing.md:83 Fenced code blocks should have a language specified
  • java/operating-applications/optimizing.md:88 Fenced code blocks should have a language specified
  • java/operating-applications/optimizing.md:93 Fenced code blocks should have a language specified
  • java/operating-applications/optimizing.md:98 Fenced code blocks should have a language specified
  • java/operating-applications/optimizing.md:103 Fenced code blocks should have a language specified

Copy link
Contributor

@BraunMatthias BraunMatthias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice docu - exactly what is required

@davidhunglam
Copy link
Contributor Author

@renejeglinsky, @smahati, we still need a review from one of you. We'd like to have this merged before the upcoming CAP Java release.

@renejeglinsky renejeglinsky merged commit ea38315 into main May 6, 2025
4 checks passed
@renejeglinsky renejeglinsky deleted the async-profiler-sec branch May 6, 2025 13:33
renejeglinsky added a commit that referenced this pull request May 6, 2025
Added section about how to use the
https://github.com/SAP/cf-cli-java-plugin to run profiling with asprof.

---------

Co-authored-by: BraunMatthias <[email protected]>
Co-authored-by: Rene Jeglinsky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants