Skip to content

Commit

Permalink
[chore](log) print query id before logging profile in be.INFO (#32922)
Browse files Browse the repository at this point in the history
  • Loading branch information
zhiqiang-hhhh authored and Doris-Extras committed Apr 10, 2024
1 parent 87f9927 commit f23a72b
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
10 changes: 8 additions & 2 deletions be/src/pipeline/pipeline_fragment_context.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -460,7 +460,11 @@ void PipelineFragmentContext::trigger_report_if_necessary() {
// _runtime_state->load_channel_profile()->compute_time_in_profile(); // TODO load channel profile add timer
_runtime_state->load_channel_profile()->pretty_print(&ss);
}
VLOG_FILE << ss.str();

VLOG_FILE << "Query " << print_id(this->get_query_id()) << " fragment "
<< this->get_fragment_id() << " instance "
<< print_id(this->get_fragment_instance_id()) << " profile:\n"
<< ss.str();
}
auto st = send_report(false);
if (!st.ok()) {
Expand Down Expand Up @@ -917,7 +921,9 @@ void PipelineFragmentContext::_close_fragment_instance() {
if (_runtime_state->load_channel_profile()) {
_runtime_state->load_channel_profile()->pretty_print(&ss);
}
LOG(INFO) << ss.str();

LOG_INFO("Query {} fragment {} instance {} profile:\n {}", print_id(this->_query_id),
this->_fragment_id, print_id(this->get_fragment_instance_id()), ss.str());
}
// all submitted tasks done
_exec_env->fragment_mgr()->remove_pipeline_context(
Expand Down
4 changes: 3 additions & 1 deletion be/src/pipeline/pipeline_x/pipeline_x_fragment_context.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1347,7 +1347,9 @@ void PipelineXFragmentContext::_close_fragment_instance() {
if (_runtime_state->load_channel_profile()) {
_runtime_state->load_channel_profile()->pretty_print(&ss);
}
LOG(INFO) << ss.str();

LOG_INFO("Query {} fragment {} profile:\n {}", print_id(this->_query_id),
this->_fragment_id, ss.str());
}
// all submitted tasks done
_exec_env->fragment_mgr()->remove_pipeline_context(
Expand Down

0 comments on commit f23a72b

Please sign in to comment.