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
8288663: JFR: Disabling the JfrThreadSampler commits only a partially disabled state #9199
Conversation
👋 Welcome back mgronlun! A progress list of the required criteria for merging this PR into |
Webrevs
|
@mgronlun This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be:
You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 50 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details. ➡️ To integrate this PR with the above commit message to the |
Thank you @jbachorik for reviewing. |
Thanks @egahlin for reviewing. |
/integrate |
Going to push as commit a7df5a4.
Your commit was automatically rebased without conflicts. |
/backport jdk19 |
@mgronlun Unknown command |
/help |
@mgronlun Available commands:
|
Greetings,
please review the following change set.
The problematic situation is described in the JIRA issue.
In addition, I have taken the opportunity to align "interval" with "period", since the concept is exported and used more pervasively.
Testing: jdk_jfr
Thanks
Markus
Progress
Issue
Reviewers
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk pull/9199/head:pull/9199
$ git checkout pull/9199
Update a local copy of the PR:
$ git checkout pull/9199
$ git pull https://git.openjdk.org/jdk pull/9199/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 9199
View PR using the GUI difftool:
$ git pr show -t 9199
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/9199.diff