SQL Server - проблема производительности хранимых процедур - PullRequest
0 голосов
/ 30 января 2012

У меня есть таблица, которая вставляется каждую ночь, а затем запрашивается как таблица отчетов.

Хранимая процедура, которая работает для запроса, содержит динамическую строку SQL, разбивку на страницы и две временные таблицы.

Хорошо работает в течение первой недели

Начиная со второй недели, его производительность начинает резко падать (для возврата требуется 3,5 минуты)

Я получил и запустил выводстрока динамического SQL, она значительно быстрее (2 секунды), поэтому я думаю, что это может быть связано с компилятором

Затем я провел некоторую оптимизацию, например, изменив count(*) на count(event_id), производительность сразу вернулась, но на следующее утро производительность снова снижается.

Затем я изменил select into, чтобы явно объявить временную таблицу, производительность немедленно вернулась, но на следующее утро производительность снова снизилась.

Затем я изменил объявление временной таблицы явно обратно на select into, производительность сразу вернулась, но на следующее утро performance снова снижается.

Так что я думаю, что это не имеет ничего общего с оптимизацией кода, кажется, что каждый раз, когда SP компилируется, производительность может быть лучше всего менее чем за 24 часа

IЯ думаю о ночной вставке, которая также является 24-часовым циклом, затем я обнаружил эту вещь with (nolock), которая могла заблокировать Table1

После добавления Nolock хранимая процедура работала в течение неделипосле чего у нас снова возникла та же проблема, за исключением того, что на этот раз только веб-страница, вызывающая SP, работает медленно, запуск SP из БД быстрый ...

Вот динамический sql хранимый процесс:

CREATE PROCEDURE [dbo].[fs_1_usp_query]
 @paramerter_client_id   int = null,   
 @paramerter_event_type_id  int = null,   
 @paramerter_start_date         datetime = null,   
 @paramerter_end_date           datetime = null,   
 @paramerter_page_index   int = 1,
 @paramerter_sort_direction varchar(20),   
 @paramerter_page_count   int = 30   
AS   
BEGIN       
 SET ARITHABORT ON;   
 SET NOCOUNT ON;   

 declare @sql nvarchar(max)   

 set @sql =      '

    create table #output2    
    (   
            page_index                                      int,       
                rownumber                                         int, 
            page_count                          int,    
            client_id                           int,    
            date                                            datetime,   
      )   




        --insert into #output1   
  select   
    page_count = count(event_id) over(),       
    table1.*
    into #output1'   
  set @sql = @sql + '   
   from   
    table1 table1 with (nolock)   
   inner join   
    table2 table2 with (nolock)   
   on   
    ............................
   inner join   
    table3 table3 with (nolock)   
   on   
    ............................
   inner join   
    table4 table4
   on   
    ............................
   where   
    ............................

 if (@paramerter_client_id is not null)   
  set @sql = @sql + ' and table2.client_id = @paramerter_client_id'   

 if (@paramerter_event_type_id is not null)   
  set @sql = @sql + ' and table2.event_type_id = @paramerter_event_type_id'   

 if (@paramerter_start_date is not null)   
  set @sql = @sql + ' and table2.created_date >= @paramerter_start_date'   
 if (@paramerter_end_date is not null)   
        set @sql = @sql + ' and table2.created_date <= @paramerter_end_date'   

 declare @lv_begin_index int   
 declare @lv_end_index int   
 set @lv_begin_index = ((@paramerter_page_index - 1) * @paramerter_page_count) + 1     
 set @lv_end_index = @lv_begin_index + @paramerter_page_count    

 set @sql = @sql +   ' 

 UPDATE #output1
    SET osat_rating = ''-''   
    WHERE LEFT( osat_rating , 1 ) = ''-''         

 insert into #output2 
 select    
  page_index = ' + convert(varchar, @paramerter_page_index) + ',       
     row_number() over (order by [' + @paramerter_sort_expression + '] '+ @paramerter_sort_direction + ') as rownumber, 
     #output1.* 
 from #output1

 select #output2.* 
 from #output2 
 where   
  rownumber >= ' + convert(varchar, @lv_begin_index) + '   
 and   
  rownumber < ' + convert(varchar, @lv_end_index)    '   

 set @sql = @sql + '    
 drop table #output1    
 drop table #output2  '*

Вот снимок статического SQL в попытке следовать вашим предложениям:

Where
    Column3 = Coalesce(@parameter3, Column3)
   and
    (@start_date is null or Column_created_date >= @start_date)

   and
    (@param_1 is null 
            or 
                (@param_1 not in (‘ConstantString1’, 'ConstantString2') and Column1 = @param_1)
        or 
            (@param_1 = ‘ConstantString1’ and Column1 like 'ConstantString1%')
        or 
            (@param_1 = ‘ConstantString2’ and (Column1 is null or Column1 = ''))
)
If(@parameter_sort_direction = 'DESC')
 Begin
     insert into #temp_table_result
     select     
      page_index = convert(varchar, @parameter_page_index),        
      row_number() over 
        (
            order by CASE 
                    WHEN @parameter_sort_expression = 'Column1' THEN Column1 
                    WHEN @parameter_sort_expression = 'Column2' THEN Column2 
                    WHEN @parameter_sort_expression = 'Column3' THEN Column3 
                    WHEN @parameter_sort_expression = 'Column4' THEN Column4 
                    WHEN @parameter_sort_expression = 'Column5' THEN Column5 
                    WHEN @parameter_sort_expression = 'Column6' THEN Column6 
                    WHEN @parameter_sort_expression = 'Column7' THEN Column7
                    WHEN @parameter_sort_expression = 'Column8' THEN Column8 
                END desc--CASE 
                --      WHEN @parameter_sort_direction = 'ASC' THEN asc 
                --      WHEN @parameter_sort_expression = 'DESC' THEN     desc              
                --END
        ) as rownumber,  
      #temp_table_staging.*  
     from #temp_table_staging 
 END

Ответы [ 3 ]

2 голосов
/ 30 января 2012

Вероятно, статистика, используемая запросом для создания плана, постепенно устаревает.

Подумайте об обновлении статистики каждые 6 часов в таблицах, затронутых запросом - проверьте это в среде разработчика, если это возможно.

0 голосов
/ 30 января 2012

Я предлагаю вам полностью избежать динамического SQL, вы можете заменить некоторый код, подобный этому:

     if (@paramerter_client_id is not null)   
         set @sql = @sql + ' and table2.client_id = @paramerter_client_id'  

на

and (@paramerter_client_id IS NULL OR table2.client_id = @paramerter_client_id)

Конечно, не забудьте создатьиндекс для table2.client_id!

0 голосов
/ 30 января 2012

Я предлагаю вам попробовать использовать опцию WITH RECOMPILE для обновления плана выполнения при каждом запуске sp.

И есть еще несколько приемов для оптимизации плана выполнения для таких случаев: http://msdn.microsoft.com/en-us/library/ms181714.aspx

например:

OPTIMIZE FOR

или

PARAMETERIZATION

Надеюсь, что это поможет.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...