HappenLee commented on code in PR #34720:
URL: https://github.com/apache/doris/pull/34720#discussion_r1606585188
##########
be/src/runtime/runtime_query_statistics_mgr.cpp:
##########
@@ -102,10 +103,18 @@ static Status _do_report_exec_stats_rpc(const
TNetworkAddress& coor_addr,
TReportExecStatusParams
RuntimeQueryStatiticsMgr::create_report_exec_status_params_x(
const TUniqueId& query_id,
- const std::unordered_map<int32,
std::vector<std::shared_ptr<TRuntimeProfileTree>>>&
+ std::unordered_map<int32,
std::vector<std::shared_ptr<TRuntimeProfileTree>>>
fragment_id_to_profile,
- const std::vector<std::shared_ptr<TRuntimeProfileTree>>&
load_channel_profiles,
- bool is_done) {
+ std::vector<std::shared_ptr<TRuntimeProfileTree>>
load_channel_profiles, bool is_done) {
+ // This function will clear the data of fragment_id_to_profile and
load_channel_profiles.
+ RuntimeProfile::Counter timer {TUnit::TIME_NS};
+ Defer _print_log([&]() {
+ LOG_INFO("Create report exec status params cost {}",
Review Comment:
why we need log each profile report time？
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
