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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
