[jdk17u-dev] RFR: 8315097: Rename createJavaProcessBuilder [v5]

Goetz Lindenmaier goetz at openjdk.org
Mon Oct 21 11:06:22 UTC 2024


> This backport is based on the 21 version.
> 
> A lot of files can be patched clean.
> Others had to be resolved.
> I split the change into four commits:
> 1. The hunks that applied clean.
> 2. The test files I resolved. Details below.
> 3. The parts of ProcessTools.java I had to resolve
> 4. The files I had to fix in addition so all calls to the old method are removed
> 
> *The tests resolved:*
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/compiler/ciReplay/CiReplayBase.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/compiler/codecache/CodeCacheFullCountTest.java
> Resolve, but double check on 8279573: compiler/codecache/CodeCacheFullCountTest.java fails with "RuntimeException: the value of full_count is wrong."
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/compiler/jvmci/TestEnableJVMCIProduct.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/compiler/jvmci/jdk.vm.ci.hotspot.test/src/jdk/vm/ci/hotspot/test/TestHotSpotJVMCIRuntime.java
> Omitted because 8306992: [JVMCI] mitigate more against JVMCI related OOME causing VM to exit is not in 17.
> 
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/gc/arguments/TestVerifyBeforeAndAfterGCFlags.java
> Resolved because 8270991: G1 Full GC always performs heap verification after JDK-8269295 is not in 17.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/gc/g1/TestGCLogMessages.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/gc/g1/TestLargePageUseForAuxMemory.java
> Resolved copyright only.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/gc/g1/TestMixedGCLiveThreshold.java
> Resolved copyright only.
> 
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/runtime/ClassFile/JsrRewriting.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/runtime/ClassFile/OomWhileParsingRepeatedJsr.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/runtime/ErrorHandling/SecondaryErrorTest.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/runtime/LoadClass/TestResize.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/runtime/LocalLong/LocalLongTest.java
> Resolved.
> https://github.com/openjdk/jdk21u-dev/commits/master/test/hotspot/jtreg/runtime/Monitor/SyncOnValueBasedClassTest.ja...

Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:

  Undo edit that belongs to 8321713

-------------

Changes:
  - all: https://git.openjdk.org/jdk17u-dev/pull/2960/files
  - new: https://git.openjdk.org/jdk17u-dev/pull/2960/files/dac424dd..06c65af7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=2960&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=2960&range=03-04

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk17u-dev/pull/2960.diff
  Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/2960/head:pull/2960

PR: https://git.openjdk.org/jdk17u-dev/pull/2960


More information about the jdk-updates-dev mailing list