This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new 7c5f48486a7 Fix thrift core in asan mode (#49819)
7c5f48486a7 is described below
commit 7c5f48486a7087e0c97570190c1e3e3794bc72de
Author: wangbo <wan...@selectdb.com>
AuthorDate: Wed Apr 9 10:08:57 2025 +0800
Fix thrift core in asan mode (#49819)
---
be/src/runtime/runtime_query_statistics_mgr.cpp | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/be/src/runtime/runtime_query_statistics_mgr.cpp
b/be/src/runtime/runtime_query_statistics_mgr.cpp
index eb16cbd003a..f4e30ac8ddd 100644
--- a/be/src/runtime/runtime_query_statistics_mgr.cpp
+++ b/be/src/runtime/runtime_query_statistics_mgr.cpp
@@ -71,6 +71,7 @@ static Status _do_report_exec_stats_rpc(const
TNetworkAddress& coor_addr,
try {
rpc_client->reportExecStatus(res, req);
} catch (const apache::thrift::transport::TTransportException& e) {
+#ifndef ADDRESS_SANITIZER
LOG_WARNING("Transport exception from {}, reason: {}, reopening",
PrintThriftNetworkAddress(coor_addr), e.what());
client_status = rpc_client.reopen(config::thrift_rpc_timeout_ms);
@@ -80,6 +81,10 @@ static Status _do_report_exec_stats_rpc(const
TNetworkAddress& coor_addr,
}
rpc_client->reportExecStatus(res, req);
+#else
+ return Status::RpcError("Transport exception when report query
profile, err: {}",
+ e.what());
+#endif
}
} catch (apache::thrift::TApplicationException& e) {
if (e.getType() == e.UNKNOWN_METHOD) {
@@ -427,6 +432,7 @@ void
RuntimeQueryStatisticsMgr::report_runtime_query_statistics() {
coord->reportExecStatus(res, params);
rpc_result[addr] = true;
} catch (apache::thrift::transport::TTransportException& e) {
+#ifndef ADDRESS_SANITIZER
LOG_WARNING(
"[report_query_statistics] report to fe {} failed,
reason:{}, try reopen.",
add_str, e.what());
@@ -440,6 +446,9 @@ void
RuntimeQueryStatisticsMgr::report_runtime_query_statistics() {
coord->reportExecStatus(res, params);
rpc_result[addr] = true;
}
+#else
+ std::cerr << "thrift error, reason=" << e.what();
+#endif
}
} catch (apache::thrift::TApplicationException& e) {
LOG_WARNING(
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
