From c10bbefde20ba2abbcef383a5f5aafa1604438f3 Mon Sep 17 00:00:00 2001 From: laihui Date: Sun, 16 Mar 2025 11:18:23 +0800 Subject: [PATCH] error URL does not depend on the value of _number_filtered_rows --- 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 b3af2531f15dd7..5accce0f885dd3 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 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 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()) {