Discussion :

[djalil]

Bonjour Tout le monde,

J'utilise sql server 2008, j'ai un souci sur un between de date.
Mon probleme est le suivant, j'avais une requete qui m'afficher les données entre deux dates, ces dates sont récupérées dans une table, le type des colonnes est datetime. la requete met moins d'une seconde.jusqu'à la tout va bien. Cette requete est inserer dans procedure stockées. voir req1

le besoin a evolué et maintenant les dates doivent etre passées en parametre. c'est ce que j'ai fait sur la req2, mais la requete met 1 min pour afficher le resultat pour les mêmes dates utiliser dans la req1.
quelqu'un aurait il une solution pour ameliorer mes temps de réponses


req1:

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
19
20
21
22
23
24
25
26
27
28
select  distinct sect.lg_nomparent lg_nomparent,count(distinct t.AccountId) nb,convert (nvarchar(max),ouv.lg_datededebut,112),
  COUNT(DISTINCT cr.lg_compterenduid) AS nbr_CR,convert (nvarchar(max),ouv.lg_datedefin,112) --into #J
 from Lg_compterendu  cr 
 inner JOIN  lg_systemuser_lg_compterendu AS cr_usr ON cr_usr.lg_compterenduid = cr.lg_compterenduid 
 inner JOIN  SystemUser AS usr ON usr.systemuserid = cr_usr.systemuserid
 inner join lg_lg_pole_systemuser pole_usr on pole_usr.systemuserid=usr.systemuserid
 inner join Lg_pole pole on pole_usr.lg_poleid=pole.lg_poleid
 inner join lg_marque_lg_compterendu marq_cr on marq_cr.lg_compterenduid=cr.lg_compterenduid
 inner join Lg_marque marq on marq.lg_marqueid=marq_cr.lg_marqueid
 inner join lg_contact_lg_compterendu cont_cr on cont_cr.lg_compterenduid=cont_cr.lg_compterenduid
 inner join Contact cont on cont.contactid=cont_cr.contactid
 inner join Lg_ouvertureterrainsdejeu ouv on ouv.lg_commercial=usr.systemuserid,Account acc
 INNER JOIN Lg_terraindejeu AS terrain ON terrain.lg_annonceur = acc.accountid
 inner join (select tot.AccountId,tot.lg_client from (
             select AccountId,AccountId lg_client from account a
             union 
             select v.lg_tiersdefacturation AccountId,lg_client  from Lg_tiersdefacturation v) tot) t 
             on acc.AccountId=t.lg_client
  inner join Lg_secteurdactivite sect on sect.lg_secteurdactiviteid=terrain.lg_secteurdactivite
  
  where 
 marq.lg_client=t.accountid
and cr.lg_compterenduid=cont_cr.lg_compterenduid
and terrain.lg_commercial=usr.systemuserid 
  and cont.lg_fonctionlp in( '29','19','16')
  and convert(nvarchar(max),cr.lg_daterdv,112)  BETWEEN convert (nvarchar(max),ouv.lg_datededebut,112) AND convert (nvarchar(max),ouv.lg_datedefin,112)  and pole.lg_poleid='9B5092D3-7774-DF11-9725-00505684640D'
    group by sect.lg_nomparent,convert (nvarchar(max),ouv.lg_datededebut,112),convert (nvarchar(max),ouv.lg_datedefin,112)
  HAVING      COUNT(DISTINCT cr.lg_compterenduid) >= 1

req2:

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
19
20
21
22
23
24
25
26
27
28
select  distinct sect.lg_nomparent lg_nomparent,count(distinct t.AccountId) nb,convert (nvarchar(max),ouv.lg_datededebut,112),
  COUNT(DISTINCT cr.lg_compterenduid) AS nbr_CR,convert (nvarchar(max),ouv.lg_datedefin,112) --into #J
 from Lg_compterendu  cr 
 inner JOIN  lg_systemuser_lg_compterendu AS cr_usr ON cr_usr.lg_compterenduid = cr.lg_compterenduid 
 inner JOIN  SystemUser AS usr ON usr.systemuserid = cr_usr.systemuserid
 inner join lg_lg_pole_systemuser pole_usr on pole_usr.systemuserid=usr.systemuserid
 inner join Lg_pole pole on pole_usr.lg_poleid=pole.lg_poleid
 inner join lg_marque_lg_compterendu marq_cr on marq_cr.lg_compterenduid=cr.lg_compterenduid
 inner join Lg_marque marq on marq.lg_marqueid=marq_cr.lg_marqueid
 inner join lg_contact_lg_compterendu cont_cr on cont_cr.lg_compterenduid=cont_cr.lg_compterenduid
 inner join Contact cont on cont.contactid=cont_cr.contactid
 inner join Lg_ouvertureterrainsdejeu ouv on ouv.lg_commercial=usr.systemuserid,Account acc
 INNER JOIN Lg_terraindejeu AS terrain ON terrain.lg_annonceur = acc.accountid
 inner join (select tot.AccountId,tot.lg_client from (
             select AccountId,AccountId lg_client from account a
             union 
             select v.lg_tiersdefacturation AccountId,lg_client  from Lg_tiersdefacturation v) tot) t 
             on acc.AccountId=t.lg_client
  inner join Lg_secteurdactivite sect on sect.lg_secteurdactiviteid=terrain.lg_secteurdactivite
  
  where 
 marq.lg_client=t.accountid
and cr.lg_compterenduid=cont_cr.lg_compterenduid
and terrain.lg_commercial=usr.systemuserid 
  and cont.lg_fonctionlp in( '29','19','16')
  and convert(nvarchar(max),cr.lg_daterdv,112)  BETWEEN '20090831' AND '20100830'  and pole.lg_poleid='9B5092D3-7774-DF11-9725-00505684640D'
    group by sect.lg_nomparent,convert (nvarchar(max),ouv.lg_datededebut,112),convert (nvarchar(max),ouv.lg_datedefin,112)
  HAVING      COUNT(DISTINCT cr.lg_compterenduid) >= 1

[Waldar]

Votre problème c'est que vous convertissez toutes vos données DATETIME en NVARCHAR, c'est une opération coûteuse qui en plus invalide l'utilisation d'un index.

Essayez tout simplement avec ceci :

Code :

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

and cr.lg_daterdv BETWEEN ouv.lg_datededebut AND ouv.lg_datedefin

[mikedavem]

Bonjour,

Pourquoi convertissez vous avec des nvarchar(max) ?

Je suppose que vous voulez extraire les dates dans les heures dans votre BETWEEN.

Solution avec 2008 :

Code :

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

1
2
3
...
cr.lg_daterdv BETWEEN CAST(ouv.lg_datededebut AS DATE) AND CAST(ouv.lg_datedefin AS DATE)
...

Edit : Est ce vraiment nécessaire dans votre cas d'extraire la partie date ?

++

[djalil]

Bonjour,
Merci pour vos réponse.

Mon probléme est au niveau de la requetes 2 au niveau
convert(nvarchar(max),cr.lg_daterdv,112) BETWEEN '20090831' AND '20100830'
comment dois-je réecrire cette partie pour avoir les meme temps de réponse que sur la requete 1

Merci

[djalil]

J'ai utilisé la syntaxe avec les cast, mais ça met toujours 1min

Code :

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

and cast(cr.lg_daterdv as date) BETWEEN cast('20090831' AS date) AND cast('20100830' as date)

[SQLpro]

Pourquoi voulez vous caster une date ????? Une date étant un type date laissez là comme ça !
Ce que vous faites c'est convertir une date qui est donc un type date en string pour faire des comparaisons ! C'est idiot !
Pour vos dates en paramètre, prenez deux variables de type date.

a +

[mikedavem]

On pourrait avoir le code de la procédure et un éventuel plan d'exécution ?

++

[djalil]

J'utilise le cast pour enlever l'heure.
dans ma procedure les varaibles sont definies avec le type date.
est ce qu'il faut caster la partie à droite du between

[Sergejack]

J'ai utilisé la syntaxe avec les cast, mais ça met toujours 1min

Code :

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

and cast(cr.lg_daterdv as date) BETWEEN cast('20090831' AS date) AND cast('20100830' as date)

Vous n'avez pas à caster la partie gauche de l'opérateur. (ce qui, comme il a été dit, empêche tout usage potentiel d'un index approprié).

Ça devient :

Code :

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

and cr.lg_daterdv BETWEEN cast('20090831' AS date) AND cast('20100830' as date)

Ce qui fontionnera bien mieux au pire pire mieux.

[SQLpro]

Toute création de tables temporaires :
1) oblige à des recompilations => perte du plan de requête
2) génère des opérations de disque (transaction DDL)
3) dédouble les données (donc encombre le cache, pression mémoire, au détriment des données de production...)
4) force des plans sans index (scan de tables) pour toutes les tables créé par SELECT ... INTO

Bref, c'est la pire m...

a +