Уточнение запроса к хранимой процедуре заставляет его работать очень медленно

Question

У меня очень длинная хранимая процедура с несколькими блоками логики в ней, которые вставляются в разные таблицы. Вот один такой блок

У меня следующую таблицу с уникальным ограничением на

[id] [int] IDENTITY(1,1) NOT NULL 
[data] [varchar](512) NULL 

Этот блок попыток «данные», чтобы вставить значение «данные». если это значение уникально, оно вставлено. Во всех случаях соответствующий идентификатор данных возвращается

BEGIN TRY 
    INSERT INTO Data SELECT @data; 
END TRY 
BEGIN CATCH 
END CATCH 
SET @data_id = (SELECT id FROM Data WHERE data = @data); 

Когда я включаю этот блок кода в моей первоначальной хранимой процедуры, она прекрасно работает. Однако, ради аккуратности I и DRY, я думал, что я абстрактный его к югу от процедуры, так как тот же блок называется в нескольких другом SPs

ALTER PROCEDURE [dbo].[q_Data_TryInsert] 

    @data nvarchar(512), 
    @id INT OUTPUT 

AS 
BEGIN 

    BEGIN TRY 
     INSERT INTO Data SELECT @data; 
    END TRY 
    BEGIN CATCH 
    END CATCH 
    SET @id = (SELECT id FROM Data WHERE data = @data);  

END 

тогда я называю эту абстрактную SP, как так что

EXEC [q_Data_TryInsert] @data, @data_id OUTPUT 

Отремонтированный SP замедляет весь процесс на несколько порядков, даже если код тот же.

Почему это происходит?

Answer 1

измените

INSERT INTO Data SELECT @data; 

к

INSERT INTO Data (data) 
VALUES (@data) 

И изменить

SET @data_id = (SELECT id FROM Data WHERE data = @data); 

в

SET @data_id = IDENT_CURRENT('Data') 

EDIT: , чтобы получить то, что вам нужно процедура магазина должна быть переработана таким образом

ALTER PROCEDURE [dbo].[q_Data_TryInsert] 

    @data nvarchar(512), 
    @id INT OUTPUT 

AS 
BEGIN 
    IF NOT EXISTS(SELECT id FROM Data WHERE data = @data) 
    BEGIN 
    INSERT INTO Data (data) Values (@data) 
    SET @data_id = IDENT_CURRENT('Data')  
    END 
    ELSE 
    SET @id = (SELECT id FROM Data WHERE data = @data);  

END 

Answer 2

INSERT INTO [PKvalue] ([value]) 
select 'Data6' as [value] 
where not exists (select top 1 ID from [PKvalue] where [value] = 'Data6'); 
select top 1 ID from [PKvalue] where [value] = 'Data6'; 

INSERT INTO data (data) 
select @dtata as [data] 
where not exists (select top 1 ID from [data] where [data] = @data); 
select top 1 ID from [data] where [data] = '@data; 

даже не нужна транзакция. Эта вставка - это транзакция. Даже если другая вставка произошла до выбора, вы все равно получите правильный ответ. Только удаление или обновление могут нарушить выбор. У транзакции есть накладные расходы.

Answer 3

Тест на data, экономия @id. Вставьте @data, если необходимо. Обновите @id при необходимости.

BEGIN TRANSACTION 
    DECLARE @output TABLE (id int) 

    SELECT @id = id FROM #Data WHERE data = @data 

    INSERT Data (data) 
    OUTPUT inserted.[id] INTO @output 
    SELECT @data 
    WHERE @id IS NULL 

    SELECT TOP 1 @id = id FROM @output 
COMMIT TRANSACTION