This is an automated email from the ASF dual-hosted git repository.

yiguolei 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 ebb7b646240 [chore](log) Do not print profile in be log if query comes 
from FE #40399 (#41058)
ebb7b646240 is described below

commit ebb7b646240b12f1602ea99715c47f2b5d8ae438
Author: zhiqiang <[email protected]>
AuthorDate: Sun Sep 22 17:52:08 2024 +0800

    [chore](log) Do not print profile in be log if query comes from FE #40399 
(#41058)
    
    cherry pick from #40399
---
 be/src/pipeline/pipeline_fragment_context.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/be/src/pipeline/pipeline_fragment_context.cpp 
b/be/src/pipeline/pipeline_fragment_context.cpp
index 19eb023e27e..07c8dff13c0 100644
--- a/be/src/pipeline/pipeline_fragment_context.cpp
+++ b/be/src/pipeline/pipeline_fragment_context.cpp
@@ -1663,14 +1663,14 @@ void 
PipelineFragmentContext::_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));
-    // Print profile content in info log is a tempoeray solution for stream 
load.
+    // Print profile content in info log is a tempoeray solution for stream 
load and external_connector.
     // Since stream load does not have someting like coordinator on FE, so
     // backend can not report profile to FE, ant its profile can not be shown
     // in the same way with other query. So we print the profile content to 
info log.
-    // Print profile content in log is harmful for log readability, info log 
will be
-    // full of profile content, and not just profile of stream load.
-    // We know it, but currently we do not have a cheap and good solution for 
this.
-    if (_runtime_state->enable_profile()) {
+
+    if (_runtime_state->enable_profile() &&
+        (_query_ctx->get_query_source() == QuerySource::STREAM_LOAD ||
+         _query_ctx->get_query_source() == QuerySource::EXTERNAL_CONNECTOR)) {
         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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to