commit 90b2d6787ae5c43a058a102d79aa1a944dafa479
author Ian Maxon <ian@maxons.email> Thu May 25 15:47:32 2023 -0700
committer Ian Maxon <imaxon@uci.edu> Fri May 26 21:48:49 2023 +0000
tree b4cbf949ce08e17cffc37d3d3eff3c4312cd9bb8
parent ae3c273a549a92501f47ccf8713cb2aba1beee58

[ASTERIXDB-3198] Update JRE warning

- user model changes: no
- storage format changes: no
- interface changes: no

Details:

Update the warning to allow for 11 and 17, disallow anything older,
and warn about anything newer

Change-Id: I217da04d06884d841c4a56aee3ab9815ce659da8
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17558
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Ian Maxon <imaxon@uci.edu>
Reviewed-by: Wail Alkowaileet <wael.y.k@gmail.com>

asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/ApplicationConfigurator.java

diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/ApplicationConfigurator.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/ApplicationConfigurator.java
index 98a1820..44433db 100644
--- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/ApplicationConfigurator.java
+++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/ApplicationConfigurator.java
@@ -71,19 +71,17 @@
         final String javaVersion = System.getProperty("java.version");
         LOGGER.info("Found JRE version " + javaVersion);
         String[] splits = javaVersion.split("\\.");
-        if ("1".equals(splits[0])) {
-            switch (splits[1]) {
-                case "9":
-                    LOGGER.warn("JRE version \"" + javaVersion + "\" is untested");
-                    //fall-through
-                case "8":
-                    return;
-                default:
-                    throw RuntimeDataException.create(ErrorCode.UNSUPPORTED_JRE,
-                            "a minimum version of JRE of 1.8 is required, but is currently: \"" + javaVersion + "\"");
-            }
-        } else {
-            LOGGER.warn("JRE version \"" + javaVersion + "\" is untested");
+        switch (splits[0]) {
+            //versions before 9 start with a 1, e.g. 1.8
+            case "1":
+                throw RuntimeDataException.create(ErrorCode.UNSUPPORTED_JRE,
+                        "a minimum version of JRE of 11 is required, but is currently: \"" + javaVersion + "\"");
+            case "11":
+            case "17":
+                break;
+            default:
+                LOGGER.warn("JRE version \"" + javaVersion + "\" is untested");
+                break;
         }
     }