-
Notifications
You must be signed in to change notification settings - Fork 40
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
Merge jdk:jdk-24+25 #184
base: crac
Are you sure you want to change the base?
Merge jdk:jdk-24+25 #184
Conversation
Reviewed-by: jsikstro, eosterlund, stefank
Reviewed-by: vromero, abimpoudis, mcimadamore, alanb
Reviewed-by: avu, prr
…stead of uncommon traps Reviewed-by: epeter, thartmann
Reviewed-by: liach, yzheng, rriggs
Reviewed-by: naoto, jlu
Reviewed-by: asotona, jpai, naoto
…d DataFrames from closed streams Reviewed-by: jpai
Reviewed-by: chagedorn, dfenacci, dlunden
…stream Co-authored-by: Doug Lea <[email protected]> Reviewed-by: vklang
… field Reviewed-by: darcy
…ransitive java.base; Co-authored-by: Mark Reinhold <[email protected]> Co-authored-by: Alex Buckley <[email protected]> Reviewed-by: hannesw, asotona
…rth Preview) Reviewed-by: asotona, jpai
Reviewed-by: dholmes, iklam
… MethodData on failure Reviewed-by: shade, dholmes, tschatzl
…ing is enabled. Reviewed-by: iklam, ccheung
…ot/jtreg/ProblemList.txt Reviewed-by: chagedorn, kvn
…nCollectionSetRegionClosure Reviewed-by: iwalulya, ayang, sangheki
…i module Reviewed-by: rriggs, ascarpino
…eam().read() wasn't preempted Reviewed-by: dfuchs, msheppar, syan, alanb
…with Parallel/Serial GC Reviewed-by: mli
…ecision Reviewed-by: alanb
Reviewed-by: sspitsyn, cjplummer
Reviewed-by: mcimadamore
Reviewed-by: mullan, ascarpino
… failed 8344046: Tests under cds/appcds/jvmti/redefineClasses should have @requires vm.cds Reviewed-by: iklam, matsaave
…tafactory Reviewed-by: jvernee
…ixed to support all GC Reviewed-by: stefank
…ktop module Reviewed-by: honkar, kcr
… module Reviewed-by: rriggs, ascarpino
Reviewed-by: stefank
…otocol.https after JEP 486 integration Reviewed-by: dfuchs
….URLClassLoader after JEP 486 integration Reviewed-by: alanb, yzheng
…th some WMs Reviewed-by: prr, azvegint
….Socket and java.net.ServerSocket after JEP 486 integration Reviewed-by: dfuchs, alanb, jpai
…ctionFactory Reviewed-by: liach, rriggs
…d-forwarding failures Reviewed-by: chagedorn, qamai
…ry input to at least one load/store Co-authored-by: Roberto Castañeda Lozano <[email protected]> Reviewed-by: thartmann, chagedorn
…n x64 with -Xmixed Reviewed-by: iklam
Reviewed-by: kevinw, lmesnik, cjplummer
…indows Reviewed-by: almatvee
… Swing classes Reviewed-by: psadhukhan
…n the java.desktop module Reviewed-by: kizune
… be updated Reviewed-by: alanb
…java.desktop module Reviewed-by: prr
Reviewed-by: rriggs, jlahoda, jjg
Reviewed-by: alanb, lancea, iris
Reviewed-by: cjplummer, kevinw
Co-authored-by: Fei Yang <[email protected]> Reviewed-by: mli, fyang
👋 Welcome back dcherepanov! A progress list of the required criteria for merging this PR into |
@dimitryc 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 no new commits pushed to the As you do not have Committer status in this project an existing Committer must agree to sponsor your change. Possible candidates are the reviewers of this PR (@rvansa) but any other Committer may sponsor as well. ➡️ To flag this PR as ready for integration with the above commit message, type |
Webrevs |
As a part of the merge with jdk-24+25 (with the patch for JDK-8344056), CRaC options moved from src/java.base/share/man/java.1 (deleted now) to src/java.base/share/man/java.md |
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.
LGTM, thanks!
/integrate |
Merge jdk:jdk-24+25
Progress
Reviewers
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/crac.git pull/184/head:pull/184
$ git checkout pull/184
Update a local copy of the PR:
$ git checkout pull/184
$ git pull https://git.openjdk.org/crac.git pull/184/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 184
View PR using the GUI difftool:
$ git pr show -t 184
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/crac/pull/184.diff
Using Webrev
Link to Webrev Comment