From ab1061e3f854ad32367d09444f43e3a6560be3dd Mon Sep 17 00:00:00 2001 From: Socrates Date: Fri, 27 Sep 2024 16:22:44 +0800 Subject: [PATCH] [fix](hudi) fix fe memory leak (#41256) ## Proposed changes related pr: https://github.com/apache/doris/pull/41225 Should not set disable.cache to true, this will result in the accumulation of com.aliyun.oss.ClientConfiguration. --- .../datasource/hive/HiveMetaStoreClientHelper.java | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HiveMetaStoreClientHelper.java b/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HiveMetaStoreClientHelper.java index db6019eda97eef..97032467cec765 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HiveMetaStoreClientHelper.java +++ b/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HiveMetaStoreClientHelper.java @@ -67,7 +67,6 @@ import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; -import java.net.URI; import java.security.PrivilegedExceptionAction; import java.time.LocalDateTime; import java.time.ZoneId; @@ -829,15 +828,6 @@ public static T ugiDoAs(Configuration conf, PrivilegedExceptionAction act public static HoodieTableMetaClient getHudiClient(HMSExternalTable table) { String hudiBasePath = table.getRemoteTable().getSd().getLocation(); Configuration conf = getConfiguration(table); - if (LOG.isDebugEnabled()) { - LOG.debug("try setting 'fs.xxx.impl.disable.cache' to true for hudi's base path: {}", hudiBasePath); - } - URI hudiBasePathUri = URI.create(hudiBasePath); - String scheme = hudiBasePathUri.getScheme(); - if (!Strings.isNullOrEmpty(scheme)) { - // Avoid using Cache in Hadoop FileSystem, which may cause FE OOM. - conf.set("fs." + scheme + ".impl.disable.cache", "true"); - } return HadoopUGI.ugiDoAs(AuthenticationConfig.getKerberosConfig(conf), () -> HoodieTableMetaClient.builder().setConf(conf).setBasePath(hudiBasePath).build()); }