From 10f73c2c67f574a823a7e3eec61dd670ab2ed041 Mon Sep 17 00:00:00 2001 From: hui lai Date: Mon, 17 Mar 2025 10:47:00 +0800 Subject: [PATCH] [fix](multi table load) error URL does not depend on the value of _number_filtered_rows (#49111) ### What problem does this PR solve? The error URL does not depend on the value of _number_filtered_rows for it may not be correct. --- 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 08b0922a6fce54..82ebb45cea8fb4 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 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 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()) {