[Fix](JDK) Start Fe with JDK17 (apache#29658)

BePPPower · Jan 29, 2024 · 8d40209 · 8d40209
1 parent 5c817de
commit 8d40209
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/conf/fe.conf b/conf/fe.conf
@@ -39,7 +39,7 @@ JAVA_OPTS="-Djavax.security.auth.useSubjectCredsOnly=false -Xss4m -Xmx8192m -XX:
 JAVA_OPTS_FOR_JDK_9="-Djavax.security.auth.useSubjectCredsOnly=false -Xss4m -Xmx8192m -XX:+UseG1GC -XX:MaxGCPauseMillis=200 -Xlog:gc*:$DORIS_HOME/log/fe.gc.log.$CUR_DATE:time -Dlog4j2.formatMsgNoLookups=true"
 
 # For jdk 16+, this JAVA_OPTS will be used as default JVM options
-JAVA_OPTS_FOR_JDK_16="-Djavax.security.auth.useSubjectCredsOnly=false -XX:+UseZGC -Xmx8192m -Xms8192m -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=$DORIS_HOME/log/ -Xlog:gc*:$DORIS_HOME/log/fe.gc.log.$CUR_DATE:time --add-opens java.base/java.util.concurrent.locks=ALL-UNNAMED --add-opens java.base/java.security=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.lang.ref=ALL-UNNAMED --add-opens java.base/java.util.regex=ALL-UNNAMED --add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.util.concurrent=ALL-UNNAMED"
+JAVA_OPTS_FOR_JDK_16="-Djavax.security.auth.useSubjectCredsOnly=false -XX:+UseZGC -Xmx8192m -Xms8192m -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=$DORIS_HOME/log/ -Xlog:gc*:$DORIS_HOME/log/fe.gc.log.$CUR_DATE:time"
 
 ##
 ## the lowercase properties are read by main program.

diff --git a/fe/fe-core/src/main/java/org/apache/doris/persist/gson/GsonUtils.java b/fe/fe-core/src/main/java/org/apache/doris/persist/gson/GsonUtils.java
@@ -126,6 +126,7 @@
 import com.google.gson.JsonParseException;
 import com.google.gson.JsonSerializationContext;
 import com.google.gson.JsonSerializer;
+import com.google.gson.ReflectionAccessFilter;
 import com.google.gson.TypeAdapter;
 import com.google.gson.TypeAdapterFactory;
 import com.google.gson.annotations.SerializedName;
@@ -300,6 +301,7 @@ public class GsonUtils {
  // Add any other adapters if necessary.
  private static final GsonBuilder GSON_BUILDER = new GsonBuilder().addSerializationExclusionStrategy(
  new HiddenAnnotationExclusionStrategy()).enableComplexMapKeySerialization()
+ .addReflectionAccessFilter(ReflectionAccessFilter.BLOCK_INACCESSIBLE_JAVA)
  .registerTypeHierarchyAdapter(Table.class, new GuavaTableAdapter())
  .registerTypeHierarchyAdapter(Multimap.class, new GuavaMultimapAdapter())
  .registerTypeAdapterFactory(new PostProcessTypeAdapterFactory())

diff --git a/fe/pom.xml b/fe/pom.xml
@@ -239,7 +239,7 @@ under the License.
  <commons-pool.version>1.5.1</commons-pool.version>
  <commons-text.version>1.10.0</commons-text.version>
  <commons-validator.version>1.7</commons-validator.version>
- <gson.version>2.8.9</gson.version>
+ <gson.version>2.10.1</gson.version>
  <guava.version>32.1.2-jre</guava.version>
  <jackson.version>2.15.2</jackson.version>
  <java-cup.version>0.11-a-czt02-cdh</java-cup.version>