Упростите вставку SQL, которая по умолчанию использует столбец NEWSEQUNETIALID()

Question

У меня есть следующая хранимая процедура вставки:

CREATE Procedure dbo.APPL_ServerEnvironmentInsert
(
    @ServerEnvironmentName varchar(50),
    @ServerEnvironmentDescription varchar(1000),
    @UserCreatedId uniqueidentifier,
    @ServerEnvironmentId uniqueidentifier OUTPUT
)
WITH RECOMPILE
AS
    -- Stores the ServerEnvironmentId.
    DECLARE @APPL_ServerEnvironment TABLE (ServerEnvironmentId uniqueidentifier)

    -- If @ServerEnvironmentId was not supplied.
    IF (@ServerEnvironmentId IS NULL)
    BEGIN
        -- Insert the data into the table.
        INSERT INTO APPL_ServerEnvironment WITH(TABLOCKX)
        (
                ServerEnvironmentName,
                ServerEnvironmentDescription,
                DateCreated,
                UserCreatedId
        )
        OUTPUT Inserted.ServerEnvironmentId INTO @APPL_ServerEnvironment
        VALUES
        (
                @ServerEnvironmentName,
                @ServerEnvironmentDescription,
                GETDATE(),
                @UserCreatedId
        )

        -- Get the ServerEnvironmentId.
        SELECT @ServerEnvironmentId = ServerEnvironmentId
        FROM @APPL_ServerEnvironment
    END
    ELSE
    BEGIN
        -- Insert the data into the table.
        INSERT INTO APPL_ServerEnvironment WITH(TABLOCKX)
        (
                ServerEnvironmentId,
                ServerEnvironmentName,
                ServerEnvironmentDescription,
                DateCreated,
                UserCreatedId
        )
        VALUES
        (
                @ServerEnvironmentId,
                @ServerEnvironmentName,
                @ServerEnvironmentDescription,
                GETDATE(),
                @UserCreatedId
        )
    END
GO

Я мог бы упростить вышеизложенное как:

CREATE Procedure dbo.APPL_ServerEnvironmentInsert
(
    @ServerEnvironmentName varchar(50),
    @ServerEnvironmentDescription varchar(1000),
    @UserCreatedId uniqueidentifier,
    @ServerEnvironmentId uniqueidentifier OUTPUT
)
WITH RECOMPILE
AS
-- Ensure @ServerEnvironmentId IS NOT NULL
SELECT ISNULL(@ServerEnvironmentId, newid())

-- Insert the data into the table.
INSERT INTO APPL_ServerEnvironment WITH(TABLOCKX)
(
    ServerEnvironmentId,
    ServerEnvironmentName,
    ServerEnvironmentDescription,
    DateCreated,
    UserCreatedId
)
VALUES
(
    @ServerEnvironmentId,
    @ServerEnvironmentName,
    @ServerEnvironmentDescription,
    GETDATE(),
    @UserCreatedId
)
GO

Но при этом я теряю улучшения производительности newsequentialid() над newid().newsequentialid() не может быть установлен в коде как newid(), он может быть предоставлен только в качестве значения по умолчанию на уровне столбца таблицы.

Любые идеи по упрощению исходного запроса, но с использованием newsequentialid()? Или исходный запрос является наиболее упрощенным решением для достижения этой цели?

Answer 1

Моя оригинальная идея была правильной. Это самое простое и удобочитаемое решение.

Answer 2

Да. Попробуйте попробовать новое заявление о слиянии. Он должен быть на 100% совместим со столбцом по умолчанию для newsequentialid(), и он приведет SQL к единому краткому выражению. Надеюсь, это поможет.

Answer 3

Так как newsequentialid() может использоваться только в качестве значения по умолчанию для столбца, вы можете изменить исходный запрос на:

• вставить только @ServerEnvironmentId если значение не было предоставлено, тем самым генерируя новый последовательный идентификатор и извлекая его из OUTPUT пункт

• затем обновить эту строку, определенную либо @ServerEnvironmentId передано изначально, или новый идентификатор, который вы только что создали, вставив "пустую строку" в вашу таблицу

Не уверен, что это будет быстрее / эффективнее - вам придется сделать некоторые измерения на этом.