pass secondary_namespace to PLPrepareCtx in spi_parse_prepare to avoid crash on trigger referencing dropped column

obdev · wyfanxiao · LINxiansheng · ob-robot · commit dd7a0906a8ea · 2026-04-10T16:16:17.000Z
Co-authored-by: wyfanxiao &lt;wyfanxiao@163.com&gt;
Co-authored-by: LINxiansheng &lt;wangzelin19961202@gmail.com&gt;
diff --git a/src/sql/ob_spi.cpp b/src/sql/ob_spi.cpp
@@ -26,6 +26,7 @@
 #include "sql/engine/expr/ob_expr_pl_integer_checker.h"
 #include "sql/engine/expr/ob_expr_lob_utils.h"
 #include "pl/ob_pl_package.h"
+#include "share/schema/ob_trigger_info.h"
 #include "sql/engine/expr/ob_expr_obj_access.h"
 #include "pl/ob_pl_exception_handling.h"
 #include "sql/dblink/ob_tm_service.h"
@@ -2081,7 +2082,16 @@ int ObSPIService::spi_parse_prepare(common::ObIAllocator &allocator,
     } else if (OB_FAIL(ob_write_string(allocator, ObString(parse_result.no_param_sql_len_, parse_result.no_param_sql_), prepare_result.route_sql_))) {
       LOG_WARN("failed to write string", K(sql), K(ret));
     } else {
-      PLPrepareCtx pl_prepare_ctx(session, NULL, false, false, false);
+      // Pass secondary_namespace for trigger packages so that SQL in trigger body
+      // is fully resolved at compile time, catching references to dropped columns
+      // early instead of crashing at runtime. For non-trigger PL, keep using simple
+      // PS protocol (NULL) to allow deferred resolution of dynamic SQL constructs.
+      pl::ObPLBlockNS *ns_for_prepare = NULL;
+      if (OB_NOT_NULL(secondary_namespace)
+          && ObTriggerInfo::is_trigger_package_id(secondary_namespace->get_package_id())) {
+        ns_for_prepare = secondary_namespace;
+      }
+      PLPrepareCtx pl_prepare_ctx(session, ns_for_prepare, false, false, false);
       SMART_VAR(PLPrepareResult, pl_prepare_result) {
         CK (OB_NOT_NULL(GCTX.sql_engine_));
         OZ (pl_prepare_result.init(session));
