From de9ff569633b1ebc1c650aa8e9a6224c10933d13 Mon Sep 17 00:00:00 2001
From: seuwebber <webber_code@163.com>
Date: Fri, 2 Dec 2022 10:40:11 +0000
Subject: [PATCH] [to #33109923]fix bug compile package

---
 src/sql/resolver/ddl/ob_alter_package_resolver.cpp | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/sql/resolver/ddl/ob_alter_package_resolver.cpp b/src/sql/resolver/ddl/ob_alter_package_resolver.cpp
index 9f2728860b..9bc3df7743 100644
--- a/src/sql/resolver/ddl/ob_alter_package_resolver.cpp
+++ b/src/sql/resolver/ddl/ob_alter_package_resolver.cpp
@@ -150,8 +150,8 @@ int ObAlterPackageResolver::analyze_package(ObPLCompiler &compiler,
         ret = OB_SUCCESS;
         break;
     }
-    error_info.collect_error_info(package_info);
   }
+  OZ (error_info.collect_error_info(package_info));
   return ret;
 }
 
@@ -247,7 +247,6 @@ int ObAlterPackageResolver::compile_package(const ObString& db_name,
         LOG_USER_WARN(OB_ERR_PACKAGE_COMPILE_ERROR, "PACKAGE",
                       db_name.length(), db_name.ptr(),
                       package_name.length(), package_name.ptr());
-        collec_error_info(package_spec_info, error_info);
       }
     }
     if (OB_FAIL(ret)) {
@@ -292,6 +291,14 @@ int ObAlterPackageResolver::compile_package(const ObString& db_name,
                                                                   routine_infos));
       }
     }
+    if (OB_SUCC(ret)) {
+      if (!(compile_body || (!compile_body && !compile_spec))) {
+        OV (OB_NOT_NULL(package_spec_info), OB_INVALID_ARGUMENT);
+        OX (pkg_arg.tenant_id_ = package_spec_info->get_tenant_id());
+        OX (pkg_arg.package_type_ = package_spec_info->get_type());
+        OX (pkg_arg.compatible_mode_ = package_spec_info->get_compatibility_mode());
+      }
+    }
   }
   // TODO: collect error info
   return ret;