[opt](config) support dynamically change hms timeout (#59051)

### What problem does this PR solve?

Now we use FE config `hive_metastore_client_timeout_second` to force
reset
`hive.metastore.client.socket.timeout`, but there are some problem:
1. config `hive_metastore_client_timeout_second` is global, can not set
for each catalog;
2. config `hive_metastore_client_timeout_second` is Mutable, but it will
not take effect because of `ThriftHMSCachedClient`；

How to improve:
keep config `hive_metastore_client_timeout_second` as the default
timeout, while user do not override
`hive.metastore.client.socket.timeout`, and make the config immutable.

Author: cambyzju

fe/fe-common/src/main/java/org/apache/doris/common/Config.java

@@ -2246,7 +2246,7 @@ public class Config extends ConfigBase {
      * The default connection timeout for hive metastore.
      * hive.metastore.client.socket.timeout
      */
-    @ConfField(mutable = true, masterOnly = false)
+    @ConfField(mutable = false, masterOnly = false)
     public static long hive_metastore_client_timeout_second = 10;
 
     /**

fe/fe-core/src/main/java/org/apache/doris/datasource/hive/ThriftHMSCachedClient.java

@@ -37,7 +37,6 @@
 import org.apache.hadoop.hive.common.ValidTxnWriteIdList;
 import org.apache.hadoop.hive.common.ValidWriteIdList;
 import org.apache.hadoop.hive.conf.HiveConf;
-import org.apache.hadoop.hive.conf.HiveConf.ConfVars;
 import org.apache.hadoop.hive.metastore.HiveMetaHookLoader;
 import org.apache.hadoop.hive.metastore.HiveMetaStoreClient;
 import org.apache.hadoop.hive.metastore.IMetaStoreClient;
@@ -98,10 +97,6 @@ public class ThriftHMSCachedClient implements HMSCachedClient {
 
     public ThriftHMSCachedClient(HiveConf hiveConf, int poolSize, ExecutionAuthenticator executionAuthenticator) {
         Preconditions.checkArgument(poolSize > 0, poolSize);
-        if (hiveConf != null) {
-            HiveConf.setVar(hiveConf, ConfVars.METASTORE_CLIENT_SOCKET_TIMEOUT,
-                    String.valueOf(Config.hive_metastore_client_timeout_second));
-        }
         this.hiveConf = hiveConf;
         this.poolSize = poolSize;
         this.isClosed = false;

fe/fe-core/src/main/java/org/apache/doris/datasource/property/metastore/HMSBaseProperties.java

@@ -30,6 +30,7 @@
 import lombok.Getter;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.hadoop.hive.conf.HiveConf;
+import org.apache.hadoop.hive.conf.HiveConf.ConfVars;
 
 import java.util.HashMap;
 import java.util.Map;
@@ -198,8 +199,11 @@ private void checkAndInit() {
         if (StringUtils.isNotBlank(hmsUserName)) {
             hiveConf.set(AuthenticationConfig.HADOOP_USER_NAME, hmsUserName);
         }
-        HiveConf.setVar(hiveConf, HiveConf.ConfVars.METASTORE_CLIENT_SOCKET_TIMEOUT,
-                String.valueOf(Config.hive_metastore_client_timeout_second));
+        if (!userOverriddenHiveConfig.containsKey(ConfVars.METASTORE_CLIENT_SOCKET_TIMEOUT.toString())) {
+            // use Config.hive_metastore_client_timeout_second as default timeout
+            HiveConf.setVar(hiveConf, HiveConf.ConfVars.METASTORE_CLIENT_SOCKET_TIMEOUT,
+                    String.valueOf(Config.hive_metastore_client_timeout_second));
+        }
         initHadoopAuthenticator();
     }
 

fe/fe-core/src/test/java/org/apache/doris/datasource/property/metastore/HMSPropertiesTest.java

@@ -134,4 +134,19 @@ public void testHmsKerberosParams() throws UserException {
         HiveHMSProperties hmsProperties = getHMSProperties(params);
         Assertions.assertNotNull(hmsProperties);
     }
+
+    @Test
+    public void testHmsTimeoutParams() throws UserException {
+        Map<String, String> params = createBaseParams();
+        // case1: normal case, use Config.hive_metastore_client_timeout_second as default
+        HiveHMSProperties hmsProperties = getHMSProperties(params);
+        HiveConf hiveConf = hmsProperties.getHiveConf();
+        Assertions.assertEquals(String.valueOf(Config.hive_metastore_client_timeout_second),
+                hiveConf.get("hive.metastore.client.socket.timeout", null));
+
+        params.put("hive.metastore.client.socket.timeout", "123");
+        hmsProperties = getHMSProperties(params);
+        hiveConf = hmsProperties.getHiveConf();
+        Assertions.assertEquals("123", hiveConf.get("hive.metastore.client.socket.timeout", null));
+    }
 }