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

dataroaring 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 9c59134b703 branch-3.0: [fix](multi table load) error URL does not 
depend on the value of _number_filtered_rows #49111 (#49130)
9c59134b703 is described below

commit 9c59134b703a58ec9fa2933b329190ccb91f5a6e
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Mon Mar 17 15:30:54 2025 +0800

    branch-3.0: [fix](multi table load) error URL does not depend on the value 
of _number_filtered_rows #49111 (#49130)
    
    Cherry-picked from #49111
    
    Co-authored-by: hui lai <lai...@selectdb.com>
---
 be/src/io/fs/multi_table_pipe.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/be/src/io/fs/multi_table_pipe.cpp 
b/be/src/io/fs/multi_table_pipe.cpp
index eb601c5d6f5..463f002596a 100644
--- a/be/src/io/fs/multi_table_pipe.cpp
+++ b/be/src/io/fs/multi_table_pipe.cpp
@@ -271,7 +271,7 @@ Status MultiTablePipe::exec_plans(ExecEnv* exec_env, 
std::vector<ExecParam> para
                     _number_loaded_rows += state->num_rows_load_success();
                     _number_filtered_rows += state->num_rows_load_filtered();
                     _number_unselected_rows += 
state->num_rows_load_unselected();
-
+                    _ctx->error_url = 
to_load_error_http_path(state->get_error_log_file_path());
                     // check filtered ratio for this plan fragment
                     int64_t num_selected_rows =
                             state->num_rows_load_total() - 
state->num_rows_load_unselected();
@@ -280,9 +280,6 @@ Status MultiTablePipe::exec_plans(ExecEnv* exec_env, 
std::vector<ExecParam> para
                                 _ctx->max_filter_ratio) {
                         *status = Status::DataQualityError("too many filtered 
rows");
                     }
-                    if (_number_filtered_rows > 0 && 
!state->get_error_log_file_path().empty()) {
-                        _ctx->error_url = 
to_load_error_http_path(state->get_error_log_file_path());
-                    }
 
                     // if any of the plan fragment exec failed, set the status 
to the first failed plan
                     if (!status->ok()) {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to