From 18bdc08688117d28bdca6ea0c33fa862cc9aacff Mon Sep 17 00:00:00 2001 From: obdev Date: Fri, 28 Jun 2024 11:53:43 +0000 Subject: [PATCH] Change the return code of the oss direct load path --- src/sql/resolver/cmd/ob_load_data_resolver.cpp | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/sql/resolver/cmd/ob_load_data_resolver.cpp b/src/sql/resolver/cmd/ob_load_data_resolver.cpp index ee33c581e6..28a1fd1c01 100644 --- a/src/sql/resolver/cmd/ob_load_data_resolver.cpp +++ b/src/sql/resolver/cmd/ob_load_data_resolver.cpp @@ -667,7 +667,12 @@ int ObLoadDataResolver::resolve_filename(ObLoadDataStmt *load_stmt, ParseNode *n ret = OB_INVALID_ARGUMENT; LOG_USER_ERROR(OB_INVALID_ARGUMENT, "file name or access key"); } else if (OB_FAIL(load_args.access_info_.set(load_args.file_name_.ptr(), storage_info.ptr()))) { - LOG_WARN("failed to set access info", K(ret)); + if (ret == OB_INVALID_BACKUP_DEST) { + ret = OB_INVALID_ARGUMENT; + LOG_USER_ERROR(OB_INVALID_ARGUMENT, "access info"); + } else { + LOG_WARN("failed to set access info", K(ret), K(load_args.file_name_), K(storage_info)); + } } else if (OB_ISNULL(file_ptr = temp_file_name.reverse_find('/'))) { ret = OB_INVALID_ARGUMENT; LOG_USER_ERROR(OB_INVALID_ARGUMENT, "file name"); @@ -708,8 +713,8 @@ int ObLoadDataResolver::resolve_filename(ObLoadDataStmt *load_stmt, ParseNode *n } } if (OB_SUCC(ret) && load_args.file_iter_.count() == 0) { - ret = OB_BACKUP_FILE_NOT_EXIST; - LOG_WARN("No matches found for pattern", K(ret), K(pattern)); + ret = OB_FILE_NOT_EXIST; + LOG_WARN("files not exists", K(ret), K(pattern)); } } } else {