1

我在 SQL Server 2008 R2 下的更新触发器有一个小问题。我已经安装了一个触发器来将更改记录到日志表中。这对我来说很好，但当我更新多行时就不行了。

CREATE TRIGGER [dbo].[TR_CompaniesUpdated]
ON  [dbo].[Companies]
AFTER UPDATE
AS 
BEGIN
SET NOCOUNT ON;
DECLARE
    @Return NVARCHAR(MAX), -- Is the JSON of the data
    @ParameterSQL VARCHAR(MAX), -- Is the select we want to retreive in JSON format

    -- Data from of the row updated
    @ID INT,
    @StatusID SMALLINT,

    -- Old data from of the row
    @StatusID2 SMALLINT

-- Declare the cursor
DECLARE InsertedCursor CURSOR FAST_FORWARD FOR
SELECT [ID], [StatusID] FROM INSERTED

-- Create the temporary table with logs so we'll need to do only one insert after this
CREATE TABLE #tempLog(
    [ID] [int] NOT NULL,
    [Data] [nvarchar](max) NOT NULL,
    [ActionID] [int] NOT NULL,
    [DtCreated] [datetime] NOT NULL,
    [CreatedBy] [int] NOT NULL)

-- Save updated values into variables
OPEN InsertedCursor
FETCH NEXT FROM InsertedCursor
INTO @ID, @StatusID

WHILE @@fetch_status = 0
BEGIN
    -- Save old values into variables
    SELECT
        @StatusID2 = [StatusID]
    FROM DELETED
    WHERE [ID] = @ID

    SELECT * INTO #tempTable
    FROM DELETED
    WHERE [ID] = @ID

    -- Build the select only to et the changed data
    SET @ParameterSQL = 'SELECT '
    IF (ISNULL(@StatusID, '') != ISNULL(@StatusID2, '')) SET @ParameterSQL = @ParameterSQL + '[StatusID],'
    IF (@ParameterSQL != 'SELECT ')
    BEGIN
        -- Update the modified date
        UPDATE [dbo].[AssessedLevelCostCentre]
        SET [DtModified] = GETDATE()
        WHERE [ID] = @ID

        SET @ParameterSQL = SUBSTRING(@ParameterSQL, 0, LEN(@ParameterSQL)) + ' FROM #tempTable'

        -- Execute the JSON function to get the result in JSON format
        EXEC [dbo].[GetJSON] @ParameterSQL, @Return = @Return OUTPUT

        -- Insert the Change Log with the old data
        INSERT INTO #tempLog
        SELECT
            [ID] AS [ID],
            @Return AS [Data],
            [ActionID] AS [ActionID],
            CASE
                WHEN [DtModified] IS NULL
                    THEN [DtCreated]
                    ELSE [DtModified]
            END AS [DtModified],
            CASE
                WHEN [ModifiedBy] IS NULL
                    THEN [CreatedBy]
                    ELSE [ModifiedBy]
            END AS [CreatedBy]
        FROM #tempTable
    END
    ELSE
    BEGIN
        UPDATE [dbo].[AssessedLevelCostCentre]
        SET [ModifiedBy] = (SELECT [ModifiedBy] FROM #tempTable)
        WHERE [ID] = @ID
    END

    DROP TABLE #tempTable

    -- Advance the Cursor
    FETCH NEXT FROM InsertedCursor
    INTO @ID, @StatusID
END

CLOSE InsertedCursor
DEALLOCATE InsertedCursor

-- Insert the Change Log with the old data
INSERT INTO [dbo].[AssessedLevelsCostCentersLogs]
SELECT *
FROM #tempLog

DROP TABLE #tempLog
END

正如您所看到的，我有一个存储过程而不是返回 JSON 数据，问题是存储过程从查询中获取数据并且我无法传递 DELETED 或 INSERTED 表，这就是我使用临时表的原因似乎可以解决问题，但添加了 ID 子句。

更新

我更新了代码，现在我有了一个游标，但我可以看到它的性能真的很慢。我需要这种方式，因为我无法更改应用程序来创建 JSON，而且我认为我这样触发会正常工作。任何人有一个想法或如何提高这个性能？

4

1 回答 1

3

Sql server 中的触发器为每个批次运行一次，因此如果您在一条语句中插入 5000 条记录，则触发器恰好启动一次。您所做的是假设它将通过将您想要的值设置为标量变量来为每个单独的记录运行，这些变量当然只能保存一个值而不是 5000。您需要重新考虑您的过程以利用加入插入和删除一次没有运行一个记录。

于 2012-06-12T20:28:02.173 回答