fix(config): address PR review feedback on bean binding

- StorageConfig: restore try-catch around parseInt/parseLong in
  readDbOption() to preserve friendly error messages with field name,
  expected type, and actual value (matches develop Storage.setIfNeeded).
- NodeConfig: align JsonRpcConfig.maxBlockFilterNum default (0 -> 50000)
  with reference.conf and develop CommonParameter default.
- reference.conf: change DNS maxMergeSize (5 -> 0) and changeThreshold
  (0.1 -> 0.0) so behavior matches develop when users omit these keys
  (develop uses hasPath guard, leaving PublishConfig defaults of 0).
- CommitteeConfig: remove redundant alias block + public setters for
  allowPBFT/pBFTExpireNum. Public setters caused ConfigBeanFactory to
  derive uppercase property names that don't match config keys, which
  previously required the alias workaround. Keep only getters + direct
  field assignment in fromConfig().

Author: vividctrlalt

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

@@ -51,10 +51,13 @@ public class CommitteeConfig {
   @Setter(lombok.AccessLevel.NONE)
   private long pBFTExpireNum = 20;
 
+  // Only getters are exposed. No public setters — ConfigBeanFactory scans public
+  // setters via reflection and would derive key "PBFTExpireNum" / "AllowPBFT"
+  // (JavaBean uppercase rule), which does not match config keys "pBFTExpireNum"
+  // / "allowPBFT" and would throw. Values are assigned to fields directly in
+  // fromConfig() below.
   public long getAllowPBFT() { return allowPBFT; }
-  public void setAllowPBFT(long v) { this.allowPBFT = v; }
   public long getPBFTExpireNum() { return pBFTExpireNum; }
-  public void setPBFTExpireNum(long v) { this.pBFTExpireNum = v; }
   private long allowTvmFreeze = 0;
   private long allowTvmVote = 0;
   private long allowTvmLondon = 0;
@@ -99,15 +102,7 @@ public class CommitteeConfig {
   public static CommitteeConfig fromConfig(Config config) {
     Config section = config.getConfig("committee");
 
-    // ConfigBeanFactory derives key names from setter methods. For setPBFTExpireNum()
-    // it expects "PBFTExpireNum" (capital P), but config.conf uses "pBFTExpireNum".
-    // Add uppercase alias so ConfigBeanFactory can find it.
-    Config aliased = section;
-    if (section.hasPath(PBFT_EXPIRE_NUM_KEY) && !section.hasPath("PBFTExpireNum")) {
-      aliased = aliased.withValue("PBFTExpireNum", section.getValue(PBFT_EXPIRE_NUM_KEY));
-    }
-
-    CommitteeConfig cc = ConfigBeanFactory.create(aliased, CommitteeConfig.class);
+    CommitteeConfig cc = ConfigBeanFactory.create(section, CommitteeConfig.class);
     // Ensure the manually-named fields get the right values from the original keys
     cc.allowPBFT = section.hasPath(ALLOW_PBFT_KEY) ? section.getLong(ALLOW_PBFT_KEY) : 0;
     cc.pBFTExpireNum = section.hasPath(PBFT_EXPIRE_NUM_KEY)

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

@@ -300,7 +300,7 @@ public void setHttpPBFTPort(int v) {
 
     private int maxBlockRange = 5000;
     private int maxSubTopics = 1000;
-    private int maxBlockFilterNum = 0;
+    private int maxBlockFilterNum = 50000;
   }
 
   @Getter

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

@@ -230,25 +230,51 @@ private static DbOptionOverride readDbOption(Config section, String key) {
           Boolean.parseBoolean(conf.get("verifyChecksums").unwrapped().toString()));
     }
     if (conf.containsKey("compressionType")) {
-      o.setCompressionType(
-          Integer.parseInt(conf.get("compressionType").unwrapped().toString()));
+      String param = conf.get("compressionType").unwrapped().toString();
+      try {
+        o.setCompressionType(Integer.parseInt(param));
+      } catch (NumberFormatException e) {
+        throwIllegalArgumentException("compressionType", Integer.class, param);
+      }
     }
     if (conf.containsKey("blockSize")) {
-      o.setBlockSize(
-          Integer.parseInt(conf.get("blockSize").unwrapped().toString()));
+      String param = conf.get("blockSize").unwrapped().toString();
+      try {
+        o.setBlockSize(Integer.parseInt(param));
+      } catch (NumberFormatException e) {
+        throwIllegalArgumentException("blockSize", Integer.class, param);
+      }
     }
     if (conf.containsKey("writeBufferSize")) {
-      o.setWriteBufferSize(
-          Integer.parseInt(conf.get("writeBufferSize").unwrapped().toString()));
+      String param = conf.get("writeBufferSize").unwrapped().toString();
+      try {
+        o.setWriteBufferSize(Integer.parseInt(param));
+      } catch (NumberFormatException e) {
+        throwIllegalArgumentException("writeBufferSize", Integer.class, param);
+      }
     }
     if (conf.containsKey("cacheSize")) {
-      o.setCacheSize(
-          Long.parseLong(conf.get("cacheSize").unwrapped().toString()));
+      String param = conf.get("cacheSize").unwrapped().toString();
+      try {
+        o.setCacheSize(Long.parseLong(param));
+      } catch (NumberFormatException e) {
+        throwIllegalArgumentException("cacheSize", Long.class, param);
+      }
     }
     if (conf.containsKey("maxOpenFiles")) {
-      o.setMaxOpenFiles(
-          Integer.parseInt(conf.get("maxOpenFiles").unwrapped().toString()));
+      String param = conf.get("maxOpenFiles").unwrapped().toString();
+      try {
+        o.setMaxOpenFiles(Integer.parseInt(param));
+      } catch (NumberFormatException e) {
+        throwIllegalArgumentException("maxOpenFiles", Integer.class, param);
+      }
     }
     return o;
   }
+
+  private static void throwIllegalArgumentException(String param, Class<?> type, String actual) {
+    throw new IllegalArgumentException(
+        String.format("[storage.properties] %s must be %s type, actual: %s.",
+            param, type.getSimpleName(), actual));
+  }
 }

common/src/main/resources/reference.conf

@@ -369,8 +369,8 @@ node {
     dnsPrivate = ""
     knownUrls = []
     staticNodes = []
-    maxMergeSize = 5
-    changeThreshold = 0.1
+    maxMergeSize = 0
+    changeThreshold = 0.0
     serverType = ""
     accessKeyId = ""
     accessKeySecret = ""