This is an automated email from the ASF dual-hosted git repository.
zouxinyi 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 db9713855bf [fix](profile)Fix load profile print by default (#31677)
db9713855bf is described below
commit db9713855bf19d8c1c58ffa4cce86f0847c2a9df
Author: Xinyi Zou <zouxiny...@gmail.com>
AuthorDate: Fri Mar 1 19:48:07 2024 +0800
[fix](profile)Fix load profile print by default (#31677)
only enable_profile is true, print profile.
---
be/src/pipeline/pipeline_fragment_context.cpp | 2 +-
be/src/pipeline/pipeline_x/pipeline_x_fragment_context.cpp | 2 +-
be/src/runtime/plan_fragment_executor.cpp | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/be/src/pipeline/pipeline_fragment_context.cpp
b/be/src/pipeline/pipeline_fragment_context.cpp
index 44533e3a596..82b18e8bfa8 100644
--- a/be/src/pipeline/pipeline_fragment_context.cpp
+++ b/be/src/pipeline/pipeline_fragment_context.cpp
@@ -902,7 +902,7 @@ void PipelineFragmentContext::_close_fragment_instance() {
_runtime_state->runtime_profile()->total_time_counter()->update(
_fragment_watcher.elapsed_time());
static_cast<void>(send_report(true));
- if (_is_report_success) {
+ if (_runtime_state->enable_profile()) {
std::stringstream ss;
// Compute the _local_time_percent before pretty_print the
runtime_profile
// Before add this operation, the print out like that:
diff --git a/be/src/pipeline/pipeline_x/pipeline_x_fragment_context.cpp
b/be/src/pipeline/pipeline_x/pipeline_x_fragment_context.cpp
index 77a63209cc6..7362af1c0a0 100644
--- a/be/src/pipeline/pipeline_x/pipeline_x_fragment_context.cpp
+++ b/be/src/pipeline/pipeline_x/pipeline_x_fragment_context.cpp
@@ -1262,7 +1262,7 @@ void PipelineXFragmentContext::_close_fragment_instance()
{
Defer defer_op {[&]() { _is_fragment_instance_closed = true; }};
_runtime_profile->total_time_counter()->update(_fragment_watcher.elapsed_time());
static_cast<void>(send_report(true));
- if (_is_report_success) {
+ if (_runtime_state->enable_profile()) {
std::stringstream ss;
// Compute the _local_time_percent before pretty_print the
runtime_profile
// Before add this operation, the print out like that:
diff --git a/be/src/runtime/plan_fragment_executor.cpp
b/be/src/runtime/plan_fragment_executor.cpp
index 0617a89bf01..c50b137ba65 100644
--- a/be/src/runtime/plan_fragment_executor.cpp
+++ b/be/src/runtime/plan_fragment_executor.cpp
@@ -624,7 +624,7 @@ void PlanFragmentExecutor::close() {
}
}
- if (_is_report_success) {
+ if (_runtime_state->enable_profile()) {
std::stringstream ss;
// Compute the _local_time_percent before pretty_print the
runtime_profile
// Before add this operation, the print out like that:
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
