2016-10-12 6 views
-1

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

У меня возникли проблемы с тем, чтобы он работал правильно. Обычно я не работаю с курсорами, но не могу найти других вариантов. Вот проблема. Я протестировал запрос по заданным критериям для отправки только одному координатору с одной записью «if @Coord_Email = 'lamez.[email protected]», где n.id =' 43422546 '". Однако запрос выполнялся более 5 минут, поэтому мне пришлось отменить его.

ALTER PROCEDURE [dbo].[sp_MZ_Coord_rpt_s9] 
AS 
BEGIN 
    DECLARE @Member_ID VARCHAR(20) 
    DECLARE Report_S9 CURSOR FOR 
     SELECT id 
     FROM name 
     WHERE status = 'a' 

    OPEN Report_S9 

    FETCH NEXT FROM Report_S9 INTO @member_ID 

    WHILE @@FETCH_STATUS = 0 
    BEGIN 
     DECLARE @Coord_ID Varchar(20) 
     DECLARE @CO_ID Varchar(20) 
     DECLARE @Coord_Name Varchar(50) 
     DECLARE @Coord_Email Varchar(50) 

     SELECT @CO_ID = ID 
     FROM Relationship 
     WHERE id = @Member_ID 

     SELECT @Coord_ID = target_id 
     FROM Relationship 
     WHERE RELATION_TYPE = 'CO' 

     SELECT @Coord_Name = FULL_NAME 
     FROM Name 
     WHERE ID = @Coord_ID 

     SELECT @Coord_Email = email 
     FROM Name 
     WHERE id = @Coord_ID 

     IF @Coord_Email <> '' 
     BEGIN 
      SELECT 
       n.id, n.CO_ID, n.FULL_NAME, a.TRANSACTION_DATE, a.UF_1, r.TARGET_ID 
      FROM name n 
      INNER JOIN activity a ON n.id = a.id 
      INNER JOIN Tops_Profile tp ON a.id = tp.ID 
      INNER JOIN Relationship r ON n.CO_ID = r.ID 
      WHERE 
       n.id = @member 
       AND UF_1 <> '' 
       AND (DATEDIFF(dd, TRANSACTION_DATE, GETDATE()) < 2) 
       AND r.RELATION_TYPE = 'co' 
      ORDER BY 
       TRANSACTION_DATE 

      EXEC msdb..sp_send_dbmail 
        @profile_name = 'TOPS.ADMIN', 
        @recipients = @Coord_Email, 
        [email protected]_copy_recipients = , 
        @subject = 'S9 Report' 
End 

      FETCH NEXT FROM Report_S9 INTO @member_ID 
    END 

    CLOSE Report_S9 
    DEALLOCAT Report_S9 

    End 

Любая помощь очень ценится.

+0

Почему вы обертываете это в транзакции? У вас нет кода для отката транзакции, и даже если вы сделали это, вам нечего откатывать. Вы также должны рассмотреть возможность изменения привычки sp_ prefix (или даже лучше никакого префикса). http://sqlperformance.com/2012/10/t-sql-queries/sp_prefix –

+0

был другим сценарием раньше. Я исправил это. есть ли что-нибудь еще, что вы предлагаете? – Messi

+0

Вот проблема. Вы заявили: «У меня проблемы с тем, чтобы он работал правильно». Что это значит? Что он делает или не делает «правильный путь». Мы можем помочь вам в том, что не так, но вы должны объяснить нам, что не так. –

ответ

1

FETCH NEXT должен быть за пределами вашей проверки. Вам нужно продолжить цикл, даже если вам нечего делать.

+0

Хороший улов. Если OP имеет строку, где email = '', то они просто создали бесконечный цикл. –

+0

есть 2 конец. был в неправильном месте. – Messi

Смежные вопросы