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

morningman pushed a commit to branch tpc_preview4-external2
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 2d4b8c81cb67d2dcdfe544dc14164b6d5aa5ba3c
Author: Mingyu Chen (Rayner) <[email protected]>
AuthorDate: Mon Dec 22 10:25:00 2025 +0800

    [opt](profile) use same metric name as OLAP SCAN NODE for FILE SCAN NODE 
(#59159)
    
    ### What problem does this PR solve?
    
    Before:
    `FILE_SCAN_OPERATOR (id=4. nereids_id=1053. table name = warehouse):`
    
    After:
    `FILE_SCAN_OPERATOR (nereids_id=1052. table_name=inventory)(id=6):`
    
    Same format as OLAP_SCAN_OPERATOR
---
 be/src/pipeline/exec/file_scan_operator.cpp | 7 ++++---
 be/src/pipeline/exec/jdbc_scan_operator.cpp | 2 +-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/be/src/pipeline/exec/file_scan_operator.cpp 
b/be/src/pipeline/exec/file_scan_operator.cpp
index c4d64abe96e..15e068b44c8 100644
--- a/be/src/pipeline/exec/file_scan_operator.cpp
+++ b/be/src/pipeline/exec/file_scan_operator.cpp
@@ -98,9 +98,10 @@ Status 
FileScanLocalState::_init_scanners(std::list<vectorized::ScannerSPtr>* sc
 }
 
 std::string FileScanLocalState::name_suffix() const {
-    return fmt::format(" (id={}. nereids_id={}. table name = {})",
-                       std::to_string(_parent->node_id()), 
std::to_string(_parent->nereids_id()),
-                       _parent->cast<FileScanOperatorX>()._table_name);
+    return fmt::format("(nereids_id={}. table_name={})" + operator_name_suffix,
+                       std::to_string(_parent->nereids_id()),
+                       _parent->cast<FileScanOperatorX>()._table_name,
+                       std::to_string(_parent->node_id()));
 }
 
 void FileScanLocalState::set_scan_ranges(RuntimeState* state,
diff --git a/be/src/pipeline/exec/jdbc_scan_operator.cpp 
b/be/src/pipeline/exec/jdbc_scan_operator.cpp
index 71602e93587..9877e053518 100644
--- a/be/src/pipeline/exec/jdbc_scan_operator.cpp
+++ b/be/src/pipeline/exec/jdbc_scan_operator.cpp
@@ -23,7 +23,7 @@
 namespace doris::pipeline {
 #include "common/compile_check_begin.h"
 std::string JDBCScanLocalState::name_suffix() const {
-    return fmt::format("(nereids_id={} . table name = {})" + 
operator_name_suffix,
+    return fmt::format("(nereids_id={}. table_name={})" + operator_name_suffix,
                        std::to_string(_parent->nereids_id()),
                        _parent->cast<JDBCScanOperatorX>()._table_name,
                        std::to_string(_parent->node_id()));


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to