2012-10-11 186 views
1

你能帮我优化我的这个查询吗?我已经用尽了方法来优化这个。这里的数据检索时间是6分钟,我想减少它。希望您能够帮助我。需要帮助来优化此查询

-- Main query 
DECLARE @ProdClass VARCHAR(10) 
,@ModelID VARCHAR(10) 
,@ServiceStartDate DATETIME 
,@ServiceEndDate DATETIME 
,@SvcID VARCHAR(10) 
,@StateID VARCHAR(10) 
,@AreaID VARCHAR(10) 

SET @ProdClass = 'WPS' 
SET @ModelID = 'BM' 
SET @ServiceStartDate = convert(datetime, '10/1/2007') 
SET @ServiceEndDate = CONVERT(DATETIME, '11/1/2007') 
SET @SvcID = '358' 
SET @StateID = 'JB' 
SET @AreaID = '' 

SELECT DISTINCT cus.MCUS_CUSID 
    ,cus.MCUS_ENAME 
    ,mod.MMOD_ENAME 
    ,rou.MROU_SERNO 
    ,svc.MSVC_SVCID AS SVC_NAME 
    ,ar.MARE_ENAME 
    ,dbo.ufn_GetLastXApptDate(rou.MROU_ROUID) AS MROU_APTDT 
    ,rou.MROU_LSVDT 
    ,(
     CASE MROU_FRSVE 
     WHEN 'Y' 
      THEN 'FREE' 
     WHEN 'N' 
      THEN 'NORMAL' 
     END 
    ) AS SERVICE_DUE_TYPE 
FROM dbo.MROU_FIL AS rou 
INNER JOIN dbo.MCUS_FIL AS cus ON rou.MROU_CUSID = cus.MCUS_CUSID 
INNER JOIN dbo.MMOD_FIL AS mod ON rou.MROU_MODID = mod.MMOD_MODID 
INNER JOIN dbo.MSVC_FIL AS svc ON rou.MROU_SVCID = svc.MSVC_SVCID 
INNER JOIN dbo.MADR_FIL AS adr ON cus.MCUS_CUSID = adr.MADR_CUSID 
INNER JOIN dbo.MSTA_FIL AS st ON adr.MADR_STAID = st.MSTA_STAID 
INNER JOIN dbo.MARE_FIL AS ar ON adr.MADR_AREID = ar.MARE_AREID 
INNER JOIN dbo.FAPT_FIL AS apt ON rou.MROU_ROUID = apt.FAPT_ROUID 
WHERE rou.MROU_STAT = 'ACTIVE' 
AND rou.MROU_CUSPF = 'MY' 
AND apt.FAPT_APTTY in ('mf','m1','m2','m3','m4','m5','m6', 'mm') 
AND ((@ProdClass = '') OR (@ProdClass <> '' AND rou.MROU_CLSID = @ProdClass)) 
AND ((@ModelID = '') OR (@ModelID <> '' AND rou.MROU_MODID = @ModelID)) 
AND (
    ((@ServiceStartDate = '') OR (@ServiceStartDate <> '' AND rou.MROU_LSVDT >= @ServiceStartDate)) 
    AND 
    ((@ServiceEndDate = '') OR (@ServiceEndDate <> '' AND rou.MROU_LSVDT <= @ServiceEndDate)) 
) 
AND ((@SvcID = '') OR (@SvcID <> '' AND svc.MSVC_SVCID = @SvcID)) 
AND ((@StateID = '') OR (@StateID <> '' AND adr.MADR_STAID = @StateID)) 
AND ((@AreaID = '') OR (@AreaID <> '' AND adr.MADR_AREID = @AreaID)) 
ORDER BY rou.MROU_LSVDT 


-- function -- 
ALTER FUNCTION [dbo].[ufn_GetLastXApptDate] 
(
@rouid numeric(18,0) 
) 
RETURNS datetime 
AS 
BEGIN 
    DECLARE @ApptDate datetime 

    SELECT @ApptDate = MAX(FAPT_APTDT) 
    FROM dbo.FAPT_FIL 
    WHERE FAPT_ROUID = @rouid 
     AND FAPT_STAT= 'X' 
     AND FAPT_APTTY in ('mf','m1','m2','m3','m4','m5','m6', 'mm') 

    RETURN @ApptDate 

END 
+1

你对表有什么指标?如果没有,从那里开始。 – CatchingMonkey

+0

实际上很多。我不知道从哪里开始。 – Musikero31

+0

在你执行连接的每个键上至少应该有索引 - 除非表非常小。 – vstrien

回答

5

SQL Server必须调用每行的标量函数。为了自由地将函数的查询作为基于集合的操作来执行,请将函数内联移动。变化:

,dbo.ufn_GetLastXApptDate(rou.MROU_ROUID) AS MROU_APTDT 

到:

,(
SELECT MAX(FAPT_APTDT) 
FROM dbo.FAPT_FIL 
WHERE FAPT_ROUID = rou.MROU_ROUID 
     AND FAPT_STAT= 'X' 
     AND FAPT_APTTY in ('mf','m1','m2','m3','m4','m5','m6', 'mm') 
) 

您的查询也像这可能从一个不同的版本不同的参数值执行计划中受益的搜索查询。在查询结束

option (recompile) 

:您可以通过添加强制每次执行一个新的查询计划。

+0

如果您想要保留此函数中的代码,请将其转换为返回:rouid,LastXApptDate的表函数。您不会得到与此答案相同的性能提升,但仍然可以重用该代码。 – JeffO

+0

@JeffO:[内嵌表值函数](http://msdn.microsoft.com/en-us/library/ms189294(v = sql.105).aspx)与内联代码一样高效。 – Andomar

+0

明白了。非常感谢!我不知道函数的调用总是会影响查询的性能。 – Musikero31

0

尝试用你的密钥对所有列创建索引在要执行你的加入在他们出现在你的查询每个表中的顺序..