Aide pour améliorer les performances de SQL Server 2008 CTE

Question

J'ai un SQL Server 2008 CTE chargé de renvoyer le Top Review pour un emplacement. Le CTE est enveloppé dans un champ UDF (Table-Valued), et joint sur le champ LocationId, donc je peux obtenir la meilleure évaluation pour chaque emplacement.

Cardinalités:

Lieu a 0- beaucoup PostLocations
PostLocation a Poster
Post dispose critique

est ici l'UDF:

CREATE FUNCTION [dbo].[Review_HighestRated_Aggregated_ByLocation] 
( 

) 
RETURNS TABLE 
AS 
RETURN 
(
    WITH [RankedLocations] AS 
    (
     SELECT  PL.LocationId, 
        R.Rating, 
        P.PostID, 
        P.UniqueUri, 
        P.Content, 
        ROW_NUMBER() OVER (PARTITION BY PL.LocationId ORDER BY R.Rating DESC, P.LocationTypeId, P.CreatedOn DESC) As ScoreRank 

     From  dbo.PostLocations As PL 
     INNER JOIN dbo.Posts As P 
     ON   P.PostId = PL.PostId 
     INNER JOIN dbo.Reviews As R 
     ON   R.PostId = P.PostId 

     WHERE  R.ReviewTypeId <> 5 
     AND   P.Content IS NOT NULL 
    ) 

    SELECT LocationId, Rating, PostID, UniqueUri, Content 
    FROM RankedLocations 
    WHERE ScoreRank = 1 
) 

Voici un exemple de la façon dont je l'utilise:

select l.LocationId, l.Name, l.UniqueUri, r.UniqueUri, r.Content 
from @Locations l -- temp table containing around 18 location ids 
inner join dbo.Review_HighestRated_Aggregated_ByLocation() r 
on l.LocationId = r.LocationId 

La requête ci-dessus prend 15 secondes à exécuter, ce qui est inacceptable. Sans la jointure à l'UDF cela prend 0 secondes.

Des idées sur comment je peux l'améliorer? Si je regarde le plan d'exécution, c'est le SORT qui prend 98% du coût d'exécution. Le coût IO/sous-arbre de cette opération est ~ 300. J'espérais que le plan d'exécution me donnerait un indice sur un indice que je pourrais créer pour améliorer le coût, mais je n'obtiens rien.

Des idées?

Answer 1

Donc, j'ai trouvé le problème de performance, et ce n'était pas le CTE, c'était comment je l'utilisais.

J'ai plusieurs tables de recherche, une en particulier pour le type de localisation (rue = 7, ville = 5, etc.).

Donc, pour garder mon SQL fluide et cohérente (et éviter les nombres magiques codés en dur), j'ai créé une fonction scalaire wrapper qui retourne la valeur evuivalent en fonction de la chaîne, par exemple:

DECLARE @Street_LocationType = [dbo].[ToLocationTypeId]('Street') 

La fonction est extrêmement simple, juste une série d'instructions CASE.

Mais, j'utilisais mon CTE comme ceci:

SELECT  a.LocationId, b.Content, b.UniqueUri 
FROM  [dbo].[Locations] a 
INNER JOIN dbo.Review_HighestRated_Aggregated_ByLocation() b -- UDF with CTE 
ON   a.LocationId = b.LocationId   
WHERE  a.LocationTypeId = @Street_LocationType 

donc je ne même utiliser sur le CTE lui-même, je l'ai utilisé comme un filtre sur la table Emplacements.

Si je change ce qui précède pour coder en dur la valeur (par exemple 7), le temps d'exécution de la procédure passe de 13 secondes à 2 secondes. Je n'ai pas compris, mais cela a résolu le problème. Je remarquais lorsque la procédure fonctionnait mal, l'opération "TRIER" dans le plan de requête avait un nombre estimé de lignes = 32 000 - ce qui correspond à peu près à tous les messages dans le système.

Après mes modifications, le nombre estimé de lignes est 1 (comme il se doit).

Activité bizarre en effet.

Answer 2

Pour convertir le CTE et l'UDF dans une vue:

DROP FUNCTION [dbo].[Review_HighestRated_Aggregated_ByLocation] 
GO 

CREATE VIEW Review_HighestRated_Aggregated_ByLocation 
AS 
SELECT LocationId, Rating, PostID, UniqueUri, Content 
FROM 
(
    SELECT  PL.LocationId, 
       R.Rating, 
       P.PostID, 
       P.UniqueUri, 
       P.Content, 
       ROW_NUMBER() OVER (PARTITION BY PL.LocationId ORDER BY R.Rating DESC, P.LocationTypeId, P.CreatedOn DESC) As ScoreRank 
    From  dbo.PostLocations As PL 
    INNER JOIN dbo.Posts As P 
    ON   P.PostId = PL.PostId 
    INNER JOIN dbo.Reviews As R 
    ON   R.PostId = P.PostId 
    WHERE  R.ReviewTypeId <> 5 
    AND   P.Content IS NOT NULL 
) RankedLocations 
WHERE ScoreRank = 1 

GO 

exemple de requête de OP révisée pour utiliser la nouvelle vue:

select l.LocationId, l.Name, l.UniqueUri, r.UniqueUri, r.Content 
from @Locations l -- temp table containing around 18 location ids 
inner join Review_HighestRated_Aggregated_ByLocation r 
on l.LocationId = r.LocationId 

Answer 3

Si votre fonction d'une valeur de table ne avez pas besoin de paramètres envisager d'utiliser un VIEW au lieu d'un UDF. Probablement, il résout le problème de performance.