检查行是否存在后主键违规

Question

问题：哪些因素会导致以下问题？

我们有一个存储过程，用于更新或将价格记录插入表中。非常直截了当。但是，有时我们会在尝试插入已经存在的行时遇到主键冲突。如您所见，我们正在检查记录是否存在，然后酌情更新或插入。

这种情况很少发生，但上一次发生时，我们在 3 分钟的时间内发生了 18 次。我们更新的行数比 18 行多得多，所以不是每次都更新。这个 SP 经常被调用。我们检查了一下，当时没有进行索引维护。调用此 SP 的应用程序只是循环通过一个队列来更新/插入这些价格，并且只有一个应用程序实例在运行。

这是在具有 3 台服务器的 2016 可用性组上运行的。

ALTER PROCEDURE [dbo].[mw_UpdatePrice] @CustTypeID INT
    ,@ID INT
    ,@Price MONEY
    ,@OldPrice MONEY
    ,@ExpirationDate DATETIME
    ,@PriceStatusID INT
    ,@PriceStatusDesc VARCHAR(80)
    ,@FreeFreightShipviaServiceLevelID INT = NULL
    ,@FreeFreightShipViaServiceLevelDescription VARCHAR(150) = NULL
    ,@FreeFreightShipViaServiceLevelRank INT = NULL
AS
BEGIN
    SET NOCOUNT ON;

    IF EXISTS (
            SELECT 1
            FROM dbo.Price
            WHERE ID = @ID
                AND CustTypeID = @CustTypeID
            )
    BEGIN
        UPDATE dbo.Price
        SET Price = @Price
            ,OldPrice = @OldPrice
            ,ExpirationDate = @ExpirationDate
            ,PriceStatusID = @PriceStatusID
            ,PriceStatusDescription = @PriceStatusDesc
            ,ServiceLevelName = @FreeFreightShipViaServiceLevelDescription
            ,ServiceLevelId = @FreeFreightShipviaServiceLevelID
            ,ServiceLevelRank = @FreeFreightShipViaServiceLevelRank
        WHERE ID = @ID
            AND CustTypeID = @CustTypeID
    END
    ELSE
    BEGIN
        INSERT dbo.Price (
            ID
            ,CustTypeID
            ,Price
            ,OldPrice
            ,ExpirationDate
            ,PriceStatusID
            ,PriceStatusDescription
            ,ServiceLevelName
            ,ServiceLevelID
            ,ServiceLevelRank
            )
        VALUES (
            @ID
            ,@CustTypeID
            ,@Price
            ,@OldPrice
            ,@ExpirationDate
            ,@PriceStatusID
            ,@PriceStatusDesc
            ,@FreeFreightShipViaServiceLevelDescription
            ,@FreeFreightShipviaServiceLevelID
            ,@FreeFreightShipViaServiceLevelRank
            )
    END
END

Answer 1

正如 cmets 所提到的，这个存储过程并发调用是不安全的 - 它对于插入和更新场景都有竞争条件。只能通过确保对存储过程的每次调用都包含在单独的事务中（在 SP 中添加 begin/end tran，或从应用程序代码启动事务）并将HOLDLOCK 应用于 SELECT 和 INSERT 语句，才能解决此问题：

SELECT 1
FROM dbo.Price WITH (UPDLOCK, HOLDLOCK)
WHERE ID = @ID
    AND CustTypeID = @CustTypeID

...

INSERT dbo.Price WITH (HOLDLOCK)
...

即使这被重构为MERGE 语句，HOLDLOCK 仍然是防止问题的必要条件。示例here。

HOLDLOCK 锁定整个表，会降低这个SP的并发吞吐量。

除了并发问题之外，如果 ANSI_NULLS 处于打开状态，可能会发生 PRIMARY KEY 违规的另一个可能原因，这可能会导致应用程序逻辑错误，其中使用 @CustTypeID 或 @ID 的 NULL 值调用 SP，导致应用程序以非预期的方式行事。例如，使用 @CustTypeID = NULL 和 @ID = 1 调用总是会导致 INSERT，这可能不是预期的行为。

编辑：

根据@DavidBrowne 添加了UPDLOCK