Remove not longer supported G1UsePreventiveGC

core/docker/default/etc/jvm.config

@@ -15,7 +15,5 @@
 # Improve AES performance for S3, etc. on ARM64 (JDK-8271567)
 -XX:+UnlockDiagnosticVMOptions
 -XX:+UseAESCTRIntrinsics
-# Disable Preventive GC for performance reasons (JDK-8293861)
--XX:-G1UsePreventiveGC
 # Reduce starvation of threads by GClocker, recommend to set about the number of cpu cores (JDK-8192647)
 -XX:GCLockerRetryAllocationCount=32

core/trino-server-rpm/src/main/resources/dist/config/jvm.config

@@ -14,7 +14,5 @@
 # Improve AES performance for S3, etc. on ARM64 (JDK-8271567)
 -XX:+UnlockDiagnosticVMOptions
 -XX:+UseAESCTRIntrinsics
-# Disable Preventive GC for performance reasons (JDK-8293861)
--XX:-G1UsePreventiveGC
 # Reduce starvation of threads by GClocker, recommend to set about the number of cpu cores (JDK-8192647)
 -XX:GCLockerRetryAllocationCount=32

docs/src/main/sphinx/installation/deployment.md

@@ -137,8 +137,6 @@ The following provides a good starting point for creating `etc/jvm.config`:
 -XX:+UnlockDiagnosticVMOptions
 -XX:+UseAESCTRIntrinsics
 -Dfile.encoding=UTF-8
-# Disable Preventive GC for performance reasons (JDK-8293861)
--XX:-G1UsePreventiveGC  
 # Reduce starvation of threads by GClocker, recommend to set about the number of cpu cores (JDK-8192647)
 -XX:GCLockerRetryAllocationCount=32
 ```
@@ -171,8 +169,7 @@ prevents Trino from starting. You can workaround this by overriding the
 temporary directory by adding `-Djava.io.tmpdir=/path/to/other/tmpdir` to the
 list of JVM options.
 
-We enable `-XX:+UnlockDiagnosticVMOptions` and `-XX:+UseAESCTRIntrinsics` to improve AES performance for S3, etc. on ARM64 ([JDK-8271567](https://bugs.openjdk.java.net/browse/JDK-8271567))  
-We disable Preventive GC (`-XX:-G1UsePreventiveGC`) for performance reasons (see [JDK-8293861](https://bugs.openjdk.org/browse/JDK-8293861))  
+We enable `-XX:+UnlockDiagnosticVMOptions` and `-XX:+UseAESCTRIntrinsics` to improve AES performance for S3, etc. on ARM64 ([JDK-8271567](https://bugs.openjdk.java.net/browse/JDK-8271567))
 We set GCLocker retry allocation count (`-XX:GCLockerRetryAllocationCount=32`) to avoid OOM too early (see [JDK-8192647](https://bugs.openjdk.org/browse/JDK-8192647))
 
 (config-properties)=

pom.xml

@@ -209,7 +209,6 @@
             -XX:+UnlockDiagnosticVMOptions
             <!-- bump from default of 2 -->
             -XX:GCLockerRetryAllocationCount=10
-            -XX:-G1UsePreventiveGC
         </air.test.jvm.additional-arguments>
 
         <air.javadoc.lint>-missing</air.javadoc.lint>