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

yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new 1822120c13c branch-2.1: [fix](multi table load) error URL does not 
depend on the value of _number_filtered_rows #49111 (#49131)
1822120c13c is described below

commit 1822120c13c50fc651fce115074de9687dbb1133
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Wed Mar 19 09:54:36 2025 +0800

    branch-2.1: [fix](multi table load) error URL does not depend on the value 
of _number_filtered_rows #49111 (#49131)
    
    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 08b0922a6fc..82ebb45cea8 100644
--- a/be/src/io/fs/multi_table_pipe.cpp
+++ b/be/src/io/fs/multi_table_pipe.cpp
@@ -269,7 +269,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();
@@ -278,9 +278,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