This is an automated email from the ASF dual-hosted git repository.
dataroaring 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 41bde7ce818 [fix](hdfs accessor) Fix hdfs accessor build error (#33605)
41bde7ce818 is described below
commit 41bde7ce81837054c64853145d5889992fe39fac
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]
