Les jointures SQL sur les champs varchar expirent

Question

J'ai une jointure qui supprime les lignes qui correspondent à une autre table, mais les champs de jointure doivent être un grand varchar (250 caractères). Je sais que ce n'est pas idéal mais je ne peux pas penser à un meilleur moyen. Voici ma question:

DELETE P 
FROM dbo.FeedPhotos AS P 
INNER JOIN dbo.ListingPhotos AS P1 ON P.photo = P1.feedImage 
INNER JOIN dbo.Listings AS L ON P.accountID = L.accountID 
WHERE P.feedID = @feedID 

Cette requête est constamment hors délai, même si il y a moins de 1000 lignes dans la table ListingPhotos.

Toute aide serait appréciée.

Answer 1

je serais probablement commencer par suppression de cette ligne, car il ne semble pas faire quoi que ce soit:

INNER JOIN dbo.Listings AS L ON P.accountID = L.accountID 

Il pourrait ne pas avoir beaucoup de lignes dans ListingPhotos, mais s'il y a beaucoup de lignes dans Listings alors la jointure ne sera pas optimisée.

Vérifiez également votre indexation, car toute jointure est forcément lente sans les index appropriés. Bien que vous devriez généralement essayer d'éviter de rejoindre des champs de caractères de toute façon, c'est généralement un signe que les données ne sont pas normalisées correctement.

Answer 2

Il suffit d'ajouter un index.

CREATE INDEX idx_feedPhotos_feedid 
    ON dbo.FeedPhotos (feedId) 

Answer 3

Je considérerais:

• réécriture à l'utilisation EXISTE. Cela arrêtera le traitement si une ligne se trouve plus fiable reposant alors sur JOIN qui peut avoir plusieurs lignes intermédiaires (qui est ce que dit Aaronaught)

• assurer que tous les types de données correspondent exactement . Toutes les différences de longueur ou de type signifieront qu'aucun index ne sera utilisé

• En parlant de cela, avez-vous un index (approximatif) sur feedid, photo et accountid?

Quelque chose comme:

DELETE 
    P 
FROM 
    dbo.FeedPhotos AS P 
WHERE 
    P.feedID = @feedID 
    AND 
    EXISTS (SELECT * FROM 
      dbo.ListingPhotos P1 
      WHERE P.photo = P1.feedImage) 
    AND 
    EXISTS (SELECT * FROM 
      dbo.Listings L 
      WHERE P.accountID = L.accountID)