fix(config): revert validateConfig, restore ARM64 silent db engine override

The validateConfig() approach caused 1273 test failures on ARM64 because
most tests use config files with db.engine="LEVELDB" and setParam() was
throwing IllegalArgumentException before any test logic could run.

Restore the original silent override in Storage.getDbEngineFromConfig()
which automatically switches to ROCKSDB on ARM64. Update the
testConfigStorageDefaults test to be architecture-aware.

Author: vividcoder

common/src/main/java/org/tron/core/config/args/Storage.java

@@ -29,6 +29,7 @@
 import org.apache.commons.lang3.StringUtils;
 import org.iq80.leveldb.CompressionType;
 import org.iq80.leveldb.Options;
+import org.tron.common.arch.Arch;
 import org.tron.common.cache.CacheStrategies;
 import org.tron.common.cache.CacheType;
 import org.tron.common.utils.DbOptionalsUtils;
@@ -89,6 +90,7 @@ public class Storage {
    * Default values of directory
    */
   private static final String DEFAULT_DB_ENGINE = "LEVELDB";
+  private static final String ROCKS_DB_ENGINE = "ROCKSDB";
   private static final boolean DEFAULT_DB_SYNC = false;
   private static final boolean DEFAULT_EVENT_SUBSCRIBE_CONTRACT_PARSE = true;
   private static final String DEFAULT_DB_DIRECTORY = "database";
@@ -173,6 +175,10 @@ public class Storage {
   private final Map<String, Sha256Hash> dbRoots = Maps.newConcurrentMap();
 
   public static String getDbEngineFromConfig(final Config config) {
+    if (Arch.isArm64()) {
+      logger.warn("Arm64 architecture detected, using RocksDB as db engine, ignore config.");
+      return ROCKS_DB_ENGINE;
+    }
     return config.hasPath(DB_ENGINE_CONFIG_KEY)
         ? config.getString(DB_ENGINE_CONFIG_KEY) : DEFAULT_DB_ENGINE;
   }

framework/src/main/java/org/tron/core/config/args/Args.java

@@ -130,20 +130,6 @@ public static void setParam(final String[] args, final String confFileName) {
 
     // 4. Init witness (depends on CLI witness flag)
     initLocalWitnesses(config, cmd);
-
-    // 5. Validate final configuration
-    validateConfig();
-  }
-
-  /**
-   * Validate the final configuration after all layers (defaults, config, CLI) are applied.
-   * Fails fast on incompatible configurations.
-   */
-  private static void validateConfig() {
-    if (Arch.isArm64() && "LEVELDB".equalsIgnoreCase(PARAMETER.storage.getDbEngine())) {
-      throw new IllegalArgumentException(
-          "LevelDB is not supported on ARM64 architecture, please use ROCKSDB");
-    }
   }
 
   /**

framework/src/test/java/org/tron/core/config/args/ArgsTest.java

@@ -324,19 +324,18 @@ public void testCliOverridesStorageConfig() {
    *
    * <p>config-test.conf defines: db.directory = "database", db.engine = "LEVELDB".
    * Without any CLI storage arguments, the Storage object should use these config values.
-   * On ARM64, LEVELDB is not supported and validateConfig() should throw.
+   * On ARM64, the silent override in Storage.getDbEngineFromConfig() forces ROCKSDB.
    */
   @Test
   public void testConfigStorageDefaults() {
-    if (Arch.isArm64()) {
-      Assert.assertThrows(IllegalArgumentException.class,
-          () -> Args.setParam(new String[] {}, TestConstants.TEST_CONF));
-    } else {
-      Args.setParam(new String[] {}, TestConstants.TEST_CONF);
+    Args.setParam(new String[] {}, TestConstants.TEST_CONF);
 
-      CommonParameter parameter = Args.getInstance();
+    CommonParameter parameter = Args.getInstance();
 
-      Assert.assertEquals("database", parameter.getStorage().getDbDirectory());
+    Assert.assertEquals("database", parameter.getStorage().getDbDirectory());
+    if (Arch.isArm64()) {
+      Assert.assertEquals("ROCKSDB", parameter.getStorage().getDbEngine());
+    } else {
       Assert.assertEquals("LEVELDB", parameter.getStorage().getDbEngine());
     }