Discussion :

[Alghero]

Bonjour,
quelqu’un sais me dire pourquoi, cette requet vas en timeout ?
Merci

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
var tracks = (from c in ctx.TrackDatas
                    orderby c.TrackDate ascending
                    where iddevicesInContraints.Contains(c.IDDevice) && c.TimeStamp <= endDate && c.TimeStamp > startDate
                    select c).ToList();

[plume13]

Hello,
Que contient iddevicesInContraints ? Combien y a-t-il d'éléments ?

[DotNetMatt]

A priori c'est soit un problème de performance lié à la structure de la requête, soit un problème de connexion... Il nous faut le message d'erreur exact, peux-tu l'indiquer ? S'il y a une InnerException merci également d'indiquer le message.

On peut déjà analyser la requête :

• Quand tu fais un Contains dans une clause WHERE, ça équivaut à faire un LIKE en SQL, ce qui est catastrophique pour les performances, et à éviter absolument. Même s'il y a un index sur la colonne, celui-ci ne sera pas utilisé.
• Si ta colonne Timestamp n'est pas indexée correctement dans la base de données, cela nuit également aux performances.

Plus il y a de volumétrie (taille des champs en question et/ou nombre de lignes) plus ces deux points dégraderont les performances.

[Alghero]

Oui , vous avez raison c'est un problème de performance, la table TrackDatas contiens plus de 80 million de ligne.
je dois transformer ma roquette, les suggestions sont bien accepté.
merci

[Alghero]

bonjour,
Quelqu'un peux me proposer une solution !
j'ai essayé d'autre vois mais sans succès.
Merci

[plume13]

Hello,

Je ne vois pas trop comment tu peux améliorer la requête, pour moi elle est basique. Le .Contains(...) est transformé en IN (...) (et non en LIKE) et utilise a priori les index (j'ai trouvé d'ailleurs un article intéressant sur l'utilisation des index par les clauses LIKE : http://myitforum.com/cs2/blogs/jnels...16/108354.aspx).
D'ailleurs y a-t-il des index sur ces colonnes ? c'est peut-être là qu'il faut creuser.
Essaie d'effectuer des tests dans SQL Server Management Studio, tu peux récupérer la requête générée comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
IQueryable<TrackDatas> query =
    from c in ctx.TrackDatas
    orderby c.TrackDate ascending
    where iddevicesInContraints.Contains(c.IDDevice)
       && c.TimeStamp <= endDate && c.TimeStamp > startDate
    select c;
var objectQuery = query as System.Data.Objects.ObjectQuery;
Trace.TraceInformation(objectQuery.ToTraceString());

Ça te permettra d'avoir le plan d'exécution et de voir là où ça pèche.

[DotNetMatt]

Le .Contains(...) est transformé en IN (...) (et non en LIKE)

Effectivement, tu as raison dans ce cas ça se transforme bien en IN.

et utilise a priori les index (j'ai trouvé d'ailleurs un article intéressant sur l'utilisation des index par les clauses LIKE : http://myitforum.com/cs2/blogs/jnels...16/108354.aspx).

Oui du coup si on passe par une clause IN, l'index est utilisé. Par contre en LIKE, il n'est utilisé que dans le cas où il y a un préfixe ou pas de wildcard (ex. LIKE 'toto%' ou LIKE 'toto').

D'ailleurs y a-t-il des index sur ces colonnes ? c'est peut-être là qu'il faut creuser.

Oui je pense aussi qu'il faut creuser dans ce sens, et voir à quoi ressemble le plan d'exécution...

[Alghero]

Excuser moi mais c'est ce que me permets de débloquer la situation pour le moment.
sachant que iddevicesInContraints contienne une dizaine de record,
et TrackDatas contienne des centaine de million de record

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
                foreach (var idg in iddevicesInContraints)
                {
                  List<Data> trackList = new List<Data>();
                  count = count + 1;
                  decimal f = idg;
                  trackList = (from c in ctx.TrackDatas
                            orderby c.TrackDate ascending
                            where c.IDDevice == f && c.TimeStamp <= endDate && c.TimeStamp > startDate
                            select c).ToList();
                  trackListFinale.AddRange(trackList);
                  if (count == vr) {
                      tracks = trackListFinale as IEnumerable<Data>;
                      break; 
                  }
 
                   return tracks
                }

merci

[plume13]

Ah... et tu obtiens ce que tu souhaites avec ce code ? Je suis perplexe étant donné que tu ne peux passer qu'une seule fois dans la boucle foreach (à cause du break; ou du return ...; )
Pour moi le problème, c'est la base. Il n'y a pas de raison que ça parte en timeout avec quelque chose de si simple, même si il y a des millions d'enregistrements.

[Alghero]

Bonjour,
Tu as bien raison, ya eu un problème dans copy e coller du code
le return tracks c'est après terminer le cicle foreach, donc après la balise
fermeture du cicle.
mais sois sure, sans la modification la requête ne fonctionne pas.
Merci