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

zclll 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 db09cef4009 [Chore](errmsg) Avoid extra error stacktrace for error in 
scanner (#56335)
db09cef4009 is described below

commit db09cef40091daaaf3356aea761f240bbe7b6d6e
Author: zclllyybb <[email protected]>
AuthorDate: Tue Sep 23 14:36:26 2025 +0800

    [Chore](errmsg) Avoid extra error stacktrace for error in scanner (#56335)
    
    some predicate will be executed in scanner. if an error raised, before
    we would always convert it to InternalError. it's not proper. not fixed.
---
 be/src/vec/exec/scan/olap_scanner.cpp | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/be/src/vec/exec/scan/olap_scanner.cpp 
b/be/src/vec/exec/scan/olap_scanner.cpp
index e3f8e06f3a6..614c09c5e3f 100644
--- a/be/src/vec/exec/scan/olap_scanner.cpp
+++ b/be/src/vec/exec/scan/olap_scanner.cpp
@@ -277,11 +277,10 @@ Status OlapScanner::open(RuntimeState* state) {
 
     auto res = _tablet_reader->init(_tablet_reader_params);
     if (!res.ok()) {
-        std::stringstream ss;
-        ss << "failed to initialize storage reader. tablet="
-           << _tablet_reader_params.tablet->tablet_id() << ", res=" << res
-           << ", backend=" << BackendOptions::get_localhost();
-        return Status::InternalError(ss.str());
+        res.append("failed to initialize storage reader. tablet=" +
+                   std::to_string(_tablet_reader_params.tablet->tablet_id()) +
+                   ", backend=" + BackendOptions::get_localhost());
+        return res;
     }
 
     // Do not hold rs_splits any more to release memory.


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

Reply via email to