2

be/src/common/stack_trace.cpp

@@ -402,12 +402,13 @@ static void toStringEveryLineImpl([[maybe_unused]] const std::string dwarf_locat
             out << " ?";
         }
 
-        if (shouldShowAddress(physical_addr)) {
-            out << " @ ";
-            writePointerHex(physical_addr, out);
-        }
+        // Do not display the stack address and file name, it is not important.
+        // if (shouldShowAddress(physical_addr)) {
+        //     out << " @ ";
+        //     writePointerHex(physical_addr, out);
+        // }
 
-        out << "  in " << (object ? object->name : "?");
+        // out << "  in " << (object ? object->name : "?");
 
         callback(out.str());
 
@@ -459,15 +460,12 @@ std::string toStringCached(const StackTrace::FramePointers& pointers, size_t off
 }
 
 std::string StackTrace::toString(int start_pointers_index) const {
-    if (start_pointers_index == 0) {
-        return toStringCached(frame_pointers, offset, size);
-    } else {
-        // Start output from the start_pointers_index position of frame pointers.
-        StackTrace::FramePointers frame_pointers_raw {};
-        std::copy(frame_pointers.begin() + start_pointers_index, frame_pointers.end(),
-                  frame_pointers_raw.begin());
-        return toStringCached(frame_pointers_raw, offset, size - start_pointers_index);
-    }
+    // Default delete the first three frame pointers, which are inside the stack_trace.cpp.
+    start_pointers_index += 3;
+    StackTrace::FramePointers frame_pointers_raw {};
+    std::copy(frame_pointers.begin() + start_pointers_index, frame_pointers.end(),
+              frame_pointers_raw.begin());
+    return toStringCached(frame_pointers_raw, offset, size - start_pointers_index);
 }
 
 std::string StackTrace::toString(void** frame_pointers_raw, size_t offset, size_t size) {

be/src/common/status.h

@@ -376,8 +376,8 @@ class [[nodiscard]] Status {
         }
 #ifdef ENABLE_STACKTRACE
         if constexpr (stacktrace && capture_stacktrace(code)) {
-            // Delete the first four frame pointers, which are inside the StackTrace and Status.
-            status._err_msg->_stack = get_stack_trace(4);
+            // Delete the first one frame pointers, which are inside the status.h
+            status._err_msg->_stack = get_stack_trace(1);
             LOG(WARNING) << "meet error status: " << status; // may print too many stacks.
         }
 #endif