Можно ли сделать этот SQL-код более эффективным?

Question

У меня есть следующий фрагмент sql, хотя он функционирует, как и ожидалось, он немного медленно возвращает результаты (медленно я говорю о 10 секундах, чтобы вернуть 1000 результатов из диапазона дат месяца). Возможно ли сделать его более эффективным и / или более быстрым? Просто добавьте следующие индексы, которые у меня есть в таблицах: -

• RecordID - уникальный ключ первичного ключа
• Отдел - Некластеризованный
• Направление - не кластеризовано
• LocalCallGroup - не кластеризовано
• ServiceProvider - не кластеризовано
• StartTime - некластеризованный
• UserID - не кластеризован
• UserLocalStartTime - Некластеризованный
• UserLocalTimeOffset - не кластеризовано
• UserNumber - не кластеризовано

Объявление переменных здесь, убрано для поста

    SET @TerminatingSQL = '
    SELECT 
        UserNumber, 
        ImageDirection = ''in'', 
        CallingNumber = CASE WHEN callingnumber IN(''Unavailable'',''Unknown'',''+44anonymous@10.81.253.12'',''0anonymous@10.81.253.12'') THEN ''Anonymous'' ELSE callingnumber END,
        CalledNumber,
        StartTime = dateadd(ms,(-datepart(ms,(startTime))),(startTime)),
        AnswerTime = dateadd(ms,(-datepart(ms,(answerTime))),(answerTime)),
        ReleaseTime =  dateadd(ms,(-datepart(ms,(releaseTime))),(releaseTime)),         
        CallDuration =  dateadd(ms,(-datepart(ms,(releaseTime))),(releaseTime)) - dateadd(ms,(-datepart(ms,(answerTime))),(answerTime)),
        TotalDuration = dateadd(ms,(-datepart(ms,(releaseTime))),(releaseTime)) - dateadd(ms,(-datepart(ms,(startTime))),(startTime)),
        terminationCause,
        recordID
    FROM
        dbo.TABLEA' + @Table +'
    WHERE
        serviceProvider IN ( SELECT serviceProvider FROM ccNumbers WHERE CRMID = ' + CONVERT(VARCHAR(10),@CrmId,103) + ')
        AND startTime between ''' + CONVERT(VARCHAR(10),@Fromdate,112) + ''' AND ''' + CONVERT(VARCHAR(10), @ToDate, 112) + '''
        AND Direction = ''terminating''
        AND (Department = ''' + @Department + ''' OR ''' + @Department + ''' = ''ALL'')
        AND (userid = ''' + @Userid + ''' OR ''' + @Userid + ''' = ''ALL'')'

    SET @OriginatingSQL = '
    SELECT 
        UserNumber, 
        ImageDirection = ''out'', 
        CallingNumber = CASE WHEN callingnumber IN(''Unavailable'',''Unknown'',''+44anonymous@10.81.253.12'',''0anonymous@10.81.253.12'') THEN ''Anonymous'' ELSE callingnumber END,
        CalledNumber,
        StartTime = dateadd(ms,(-datepart(ms,(startTime))),(startTime)),
        AnswerTime = dateadd(ms,(-datepart(ms,(answerTime))),(answerTime)),
        ReleaseTime =  dateadd(ms,(-datepart(ms,(releaseTime))),(releaseTime)),         
        CallDuration =  dateadd(ms,(-datepart(ms,(releaseTime))),(releaseTime)) - dateadd(ms,(-datepart(ms,(answerTime))),(answerTime)),
        TotalDuration = dateadd(ms,(-datepart(ms,(releaseTime))),(releaseTime)) - dateadd(ms,(-datepart(ms,(startTime))),(startTime)),
        terminationCause,
        recordID
    FROM
        dbo.TABLEA' + @Table +'
    WHERE
        serviceProvider IN ( SELECT serviceProvider FROM ccNumbers WHERE CRMID = ' + CONVERT(VARCHAR(10),@CrmId,103) + ')
        AND startTime between ''' + CONVERT(VARCHAR(10),@Fromdate,112) + ''' AND ''' + CONVERT(VARCHAR(10), @ToDate, 112) + '''
        AND Direction = ''originating''
        AND (Department = ''' + @Department + ''' OR ''' + @Department + ''' = ''ALL'')
        AND (userid = ''' + @Userid + ''' OR ''' + @Userid + ''' = ''ALL'')'

    SET @MainSelectSQL = @TerminatingSQL + ' Union '  + @OriginatingSQL

    SET @MainSQL = 'SELECT TOP (' + @PageSize + ') 
        [t1].CalledNumber,
        [t1].CallingNumber,
        [t1].UserNumber, 
        [t1].StartTime,
        [t1].AnswerTime,
        [t1].ReleaseTime,
        [t1].ImageDirection,
        [t1].CallDuration,
        [t1].TotalDuration,
        [t1].TerminationCause
    FROM (
        SELECT ROW_NUMBER() OVER (
            ORDER BY [t0].startTime) as [row_number], 
            [t0].CalledNumber,
            [t0].CallingNumber,
            [t0].UserNumber,
            [t0].StartTime,
            [t0].AnswerTime,
            [t0].ReleaseTime,
            [t0].ImageDirection,
            [t0].CallDuration,
            [t0].TotalDuration,
            [t0].TerminationCause
        FROM
            (' + @MainSelectSQL + ')  AS [t0] 
          )  AS [t1]

    WHERE [t1].[row_number] > ' + @Page + ' * ' + @PageSize +';'

    EXEC (@MainSQL)

-- Work out the total number of rows, but don't bother if we have the number already (i.e. when they keep the same parameters and just click paging.

IF  (@CurrentCount IS NULL)
    BEGIN
        DECLARE @TotalCountSQL nvarchar(4000)
        DECLARE @ParameterList NVARCHAR(4000)   

        SET @ParameterList = '@TotalCount int OUTPUT'
        SET @TotalCountSQL = 'SELECT @TotalCount = COUNT(recordId) FROM (' + @MainSelectSQL + ') as a'

        EXEC SP_EXECUTESQL @TotalCountSQL,@ParameterList,@TotalCount=@TotalCount OUTPUT
    END
ELSE
    BEGIN
        SET @TotalCount = @CurrentCount;
    END
END

Answer 1

Что нужно улучшить

• UNION ALL удалит подразумеваемый DISTINCT. Столбец ImageDirection гарантирует, что в любом случае нет перекрытий, поэтому UNION добавляет дополнительный шаг в план

• Если у вас есть ROW_NUMBER(), добавьте COUNT(*) OVER (), чтобы получить общее количество записей. Это устраняет необходимость второго звонка

Мысль:

• У вас есть динамические имена таблиц, которые требуют такой ужасной конкатенации?
  Кроме этого, я не вижу необходимости в динамическом SQL

• Подумайте об использовании временной таблицы для упорядочения результатов, чтобы упростить сложность

Answer 2

Если вам нужен только динамический SQL из-за имен таблиц переменных, рассмотрите возможность использования только динамического SQL для извлечения необходимых данных во временную таблицу, а затем запишите весь ваш SQL как обычный SQL (не динамический) с использованием временной таблицыв качестве источника данных.

DECLARE @Table_Name VARCHAR(255) = 'Table_2011_11'

CREATE TABLE #temp (
    Number INT,
    Name VARCHAR(64)
)

DECLARE @sql VARCHAR(512) = 'INSERT INTO #temp(Number) SELECT UserNumber FROM ' + @Table_Name
exec sp_sqlexec @sql

select * from #temp