This is an automated email from the ASF dual-hosted git repository. dataroaring pushed a commit to branch branch-4.0-preview in repository https://gitbox.apache.org/repos/asf/doris.git
commit b95af21904d87e4a3b3132f29d45200598e6dd00 Author: plat1ko <[email protected]> AuthorDate: Sat Apr 13 10:36:35 2024 +0800 [fix](hdfs accessor) Fix hdfs accessor build error (#33605) --- be/src/cloud/cloud_storage_engine.cpp | 6 +++++- cloud/src/recycler/hdfs_accessor.cpp | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/be/src/cloud/cloud_storage_engine.cpp b/be/src/cloud/cloud_storage_engine.cpp index d3f4c05180c..c95998ecba1 100644 --- a/be/src/cloud/cloud_storage_engine.cpp +++ b/be/src/cloud/cloud_storage_engine.cpp @@ -275,7 +275,11 @@ void CloudStorageEngine::sync_storage_vault() { return; } - CHECK(!vault_infos.empty()) << "no s3 infos"; + if (vault_infos.empty()) { + LOG(WARNING) << "no storage vault info"; + return; + } + for (auto& [id, vault_info] : vault_infos) { auto fs = get_filesystem(id); auto st = (fs == nullptr) diff --git a/cloud/src/recycler/hdfs_accessor.cpp b/cloud/src/recycler/hdfs_accessor.cpp index 96461d7bb68..ece196baa4f 100644 --- a/cloud/src/recycler/hdfs_accessor.cpp +++ b/cloud/src/recycler/hdfs_accessor.cpp @@ -91,7 +91,7 @@ private: if (conf.has_hdfs_kerberos_principal()) { kerberos_login = true; - hdfsBuilderSetPrincipal(hdfs_builder_, conf.hdfs_kerberos_keytab().c_str()); + hdfsBuilderSetPrincipal(hdfs_builder_, conf.hdfs_kerberos_principal().c_str()); } else if (conf.has_user()) { hdfsBuilderSetUserName(hdfs_builder_, conf.user().c_str()); #ifdef USE_HADOOP_HDFS --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
