This is an automated email from the ASF dual-hosted git repository.

zykkk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new f3493e041ad [fix](build) Skip some hdfsBuilder functions when 
USE_HADOOP_HDFS support is not enabled (#49529)
f3493e041ad is described below

commit f3493e041ad0f7e5916eeb3cf88fd8eca003c6b7
Author: zy-kkk <zhongy...@gmail.com>
AuthorDate: Mon Mar 31 10:43:30 2025 +0800

    [fix](build) Skip some hdfsBuilder functions when USE_HADOOP_HDFS support 
is not enabled (#49529)
    
    We should check whether the USE_HADOOP_HDFS macro is enabled before
    deciding whether to use the hdfsBuilderSetKerb5Conf and
    hdfsBuilderSetKeyTabFile functions. Otherwise, compilation may fail in
    some environments that do not have HADOOP_HDFS compilation dependencies,
    such as MacOS.
---
 be/src/io/hdfs_builder.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/be/src/io/hdfs_builder.cpp b/be/src/io/hdfs_builder.cpp
index ed7fb0a34b4..cb9c75fb29d 100644
--- a/be/src/io/hdfs_builder.cpp
+++ b/be/src/io/hdfs_builder.cpp
@@ -227,9 +227,11 @@ Status create_hdfs_builder(const THdfsParams& hdfsParams, 
const std::string& fs_
         builder->kerberos_login = true;
         builder->hdfs_kerberos_principal = hdfsParams.hdfs_kerberos_principal;
         builder->hdfs_kerberos_keytab = hdfsParams.hdfs_kerberos_keytab;
-        hdfsBuilderSetKerb5Conf(builder->get(), 
doris::config::kerberos_krb5_conf_path.c_str());
         hdfsBuilderSetPrincipal(builder->get(), 
builder->hdfs_kerberos_principal.c_str());
+#ifdef USE_HADOOP_HDFS
+        hdfsBuilderSetKerb5Conf(builder->get(), 
doris::config::kerberos_krb5_conf_path.c_str());
         hdfsBuilderSetKeyTabFile(builder->get(), 
builder->hdfs_kerberos_keytab.c_str());
+#endif
         hdfsBuilderConfSetStr(builder->get(), 
"hadoop.kerberos.keytab.login.autorenewal.enabled",
                               "true");
         // RETURN_IF_ERROR(builder->set_kerberos_ticket_cache());


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to