Skip to content
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

[23.0] Add JFR threshold checking before event emission #575

Merged

Conversation

roberttoyonaga
Copy link
Collaborator

Summary

This PR backports threshold checking from 23.1 to 23.0. Specified duration thresholds will be checked before emitting events wherever thresholds are supported.

Original PR: oracle#6964
Oracle internal PR: oracle#7066

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 18, 2023
@roberttoyonaga roberttoyonaga added backport affects/23.0 and removed OCA Verified All contributors have signed the Oracle Contributor Agreement. labels Sep 18, 2023
Copy link
Collaborator

@zakkak zakkak left a comment

Choose a reason for hiding this comment

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

@roberttoyonaga please use git cherry-pick -x for the backports, so that we can map each commit to the one it is being backported from.

I also see some changes not being backported, e.g. oracle@d4a8436#diff-2b7c6208cd16f550c9422b6a4a25cabbb98034543abee1262f531edc253faa02

roberttoyonaga and others added 6 commits September 19, 2023 09:54
(cherry picked from commit 7c28a9b)
(cherry picked from commit 071a86d)
(cherry picked from commit 5cd7565)
(cherry picked from commit b15e6e0)
@roberttoyonaga roberttoyonaga force-pushed the jfr-backport-threshold-23.0 branch from 212e840 to a4b52d1 Compare September 19, 2023 13:56
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 19, 2023
@roberttoyonaga roberttoyonaga marked this pull request as ready for review September 19, 2023 14:24
@zakkak
Copy link
Collaborator

zakkak commented Sep 20, 2023

@roberttoyonaga now that #577 is merged could you please change the build type in the mandrel.yml workflow to mandrel-source-nolocalmvn and keep the @default pinning?

You can do so by adding the following line below with: in each job:

     build-type: "mandrel-source-nolocalmvn"

Copy link
Collaborator

@zakkak zakkak left a comment

Choose a reason for hiding this comment

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

LGTM, once the CI is happy it's OK to merge

@zakkak zakkak requested review from jerboaa and removed request for jerboaa September 21, 2023 08:47
@zakkak
Copy link
Collaborator

zakkak commented Sep 21, 2023

CI failures will be fixed with quarkusio/quarkus#36058, not related with this PR.

@zakkak zakkak merged commit 99cb157 into graalvm:mandrel/23.0 Sep 21, 2023
@zakkak zakkak mentioned this pull request Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/23.0 backport OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants