Oracle存储过程转Snowflake JS存储过程实现方案合理性咨询

你当前的转换实现不是最优方案，存在多个功能错误、安全风险以及可优化空间，具体问题如下：

• 功能错误：原Oracle逻辑插入的表是ABCD，转换后代码误写为ABC；异常分支原逻辑插入空字符串''，转换后代码写入的是字符串'NULL'，和原逻辑完全不符
• 安全风险：直接将输入参数拼接进SQL文本，存在严重的SQL注入漏洞
• 语法错误：Snowflake JavaScript存储过程中引用入参不需要{$XXX}格式，直接使用大写的参数名即可，原有写法会直接执行失败
• 冗余设计：简单的插入+异常捕获逻辑完全不需要用JavaScript存储过程实现，额外增加了执行开销

最优实现方案 #

优先使用Snowflake原生SQL存储过程，性能更高、逻辑更简洁，和原Oracle存储过程行为100%对齐：

CREATE OR REPLACE PROCEDURE ABC(
    LOGGED_USER STRING,
    REQUEST_DATE STRING,
    PAGE STRING,
    ACTIVITY STRING,
    PAYLOAD_REQUEST STRING,
    CHANGED_USER STRING
)
LANGUAGE SQL
AS
$$
BEGIN
    INSERT INTO ABCD (
        LOGGED_USER,
        request_date,
        page,
        activity,
        payload_request,
        CHANGED_USER
    ) VALUES (
        :LOGGED_USER,
        TO_DATE(:REQUEST_DATE, 'MM/DD/YYYY HH24:MI:SS'),
        :PAGE,
        :ACTIVITY,
        :PAYLOAD_REQUEST,
        :CHANGED_USER
    );
EXCEPTION
    WHEN OTHER THEN
        INSERT INTO ABCD (
            LOGGED_USER,
            request_date,
            page,
            activity,
            payload_request,
            CHANGED_USER
        ) VALUES (
            :LOGGED_USER,
            '',
            :PAGE,
            :ACTIVITY,
            :PAYLOAD_REQUEST,
            :CHANGED_USER
        );
END;
$$;

优化说明 #

• 完全规避SQL注入风险：使用原生参数绑定方式传递输入值，不需要拼接SQL
• 功能完全对齐原Oracle逻辑：表名、异常分支的赋值逻辑和原实现无差异
• 执行效率更高：没有JavaScript层的额外开销，直接在SQL引擎执行
• 可读性更强：代码结构和原Oracle存储过程高度相似，后续维护成本极低

注：如果ABCD表的REQUEST_DATE字段为DATE类型，空字符串''在Snowflake中会自动转换为NULL存储，和原Oracle的存储行为一致。

内容的提问来源于stack exchange，提问作者Sidh