存储过程首步执行失败却持续运行的问题排查与修复

• 错误未被捕获导致后续代码继续执行：当动态SQL执行插入操作触发NOT NULL约束错误时，SQL Server默认不会终止整个存储过程（除非是严重级别≥20的致命错误），会继续执行后续的UPDATE transferlog SET LastUpdate = GetDate()语句，导致失败的表也被更新了LastUpdate。
• 循环未被中断：即使第一个表插入失败，代码依然会执行SELECT @recordID = MIN(id) FROM @transferTable WHERE id > @recordID获取下一个表的ID，继续循环流程，最终所有表的LastAttempt和LastUpdate都被更新。

使用TRY/CATCH块包裹插入操作及成功更新逻辑，确保只有插入成功时才更新LastUpdate，同时可以在CATCH块中记录错误信息，按需控制循环是否继续。

修改后的代码框架如下：

SELECT @recordID = MIN(id) FROM @transferTable

WHILE @recordID IS NOT NULL
BEGIN
    -- 更新尝试记录
    UPDATE transferlog
    SET LastAttempt = GetDate()
    WHERE id = @recordID;

    -- 先获取当前循环对应的表名（从@transferTable中查询）
    SELECT @tableName = tableName FROM @transferTable WHERE id = @recordID;

    BEGIN TRY
        -- 执行插入操作，用QUOTENAME避免SQL注入和特殊表名问题
        SET @SQLCommand = 'Insert into Azure.dbo.' + QUOTENAME(@tableName) + ' SELECT * FROM Local.dbo.' + QUOTENAME(@tableName)
        PRINT 'about to run sql command for table: ' + @tableName
        EXECUTE (@SQLCommand)
        PRINT 'ran sql successfully for table: ' + @tableName

        -- 仅插入成功时更新成功记录
        UPDATE transferlog
        SET LastUpdate = GetDate()
        WHERE id = @recordID;
    END TRY
    BEGIN CATCH
        -- 捕获错误，记录错误信息到日志表（需提前创建transferErrorLog表）
        INSERT INTO transferErrorLog (recordID, TableName, ErrorMessage, ErrorTime)
        VALUES (@recordID, @tableName, ERROR_MESSAGE(), GETDATE())

        PRINT 'Failed to transfer table: ' + @tableName + ', Error: ' + ERROR_MESSAGE()
        -- 可选：如果遇到错误要终止整个传输流程，取消下面的注释
        -- BREAK;
    END CATCH

    SELECT @recordID = MIN(id) FROM @transferTable WHERE id > @recordID
END

关键说明：

• TRY/CATCH块：将插入操作和成功更新逻辑放在TRY块中，只有当TRY块内所有语句执行成功时，才会执行更新LastUpdate的操作。如果插入出错，直接跳转到CATCH块，跳过LastUpdate的更新。
• QUOTENAME函数：动态拼接表名时用该函数避免SQL注入风险，同时处理表名包含特殊字符的情况。
• 错误日志：在CATCH块中记录错误详情，方便后续排查问题。
• 循环控制：如果需要遇到错误就终止整个传输流程，可在CATCH块中添加BREAK语句；如果希望跳过错误表继续传输其他表，保留当前逻辑即可。

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