[ASE][T-SQL]Procédure trop longue

Version imprimable

Bonjour,

je débute dans les bases de données et encore plus sur SYBASE.

Voici une partie de ma procédure

Code:

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
create table #LISTE_CLIENTS
	(
		CLI_ID 			numeric(16),
		CLI_ORIGINE 		char(1),
		ANNEE			int,
		MOIS			int,
		PGE_Q_LBL_PRODUIT	varchar(32),
		PSU_CODE		char(4),
		FTP_CODE		char(4),
		FAC_CA	money,
		FAC_VOL	numeric(30,2)
	)
	create index PK_MKT_LISTE_CLIENTS_id1 on #LISTE_CLIENTS (CLI_ID)
	create index PK_MKT_LISTE_CLIENTS_id2 on #LISTE_CLIENTS (CLI_ORIGINE)
	create index PK_MKT_EFFEC_PREC_id3 on #EFFEC_PREC (FTP_CODE)
	create index PK_MKT_EFFEC_PREC_id4 on #EFFEC_PREC (PSU_CODE)
 
	Insert into #LISTE_CLIENTS
	select 
		distinct 
		T1.CLI_ID,
		T1.CLI_ORIGINE,
		T2.FAC_ANNEE,
		T2.FAC_MOIS,
		case when T3.PGE_Q_LBL_PRODUIT is NULL or T3.PGE_Q_LBL_PRODUIT='' then "NON IDENTIFIE" else T3.PGE_Q_LBL_PRODUIT end,
		case when T3.PSU_CODE is NULL or T3.PSU_CODE='' then "N.I" else T3.PSU_CODE end,
		case when T3.FTP_CODE is NULL or T3.FTP_CODE='' then "N.I" else T3.FTP_CODE end,
		sum(T2.FAC_CA),
		sum(T2.FAC_VOL)
	from 
		dm_Horus_exp.dbo.MS_CLIENT T1,
		dm_Horus_exp.dbo.MS_FACTURE T2,
		dm_Horus_exp.dbo.MS_PRESTATION T3,
		dm_Horus_exp.dbo.MS_FAC_PRESTA T4
	where
		T3.PSU_CODE in ('THEM','GEO','CFCA','CCBK','CTIN','CSIM','CNET','CSOF','COPT','CCSC','AFFI','CPL','CPL2','FGPP') or T3.FTP_CODE in ('ADR','MAIL','GEO')	
	                and T1.CLI_ID=T2.CLI_ID
		and T2.FAC_NO=T4.FAC_NO
		and T4.PGE_ID=T3.PGE_ID
		and datepart(yy, T2.FAC_DATE)=2006
		--and datepart(yy, T2.CTR_DATE_DISTRI_FIN)=@AnneeRef
	group by T1.CLI_ID, T1.CLI_ORIGINE, T2.FAC_ANNEE, T2.FAC_MOIS, T3.FTP_CODE, T3.PGE_Q_LBL_PRODUIT, T3.PSU_CODE

Ma proc marchait très bien, mais depuis que j'ai rajouté cette restriction dans le where

Code:

T3.PSU_CODE in ('THEM','GEO','CFCA','CCBK','CTIN','CSIM','CNET','CSOF','COPT','CCSC','AFFI','CPL','CPL2','FGPP') or T3.FTP_CODE in ('ADR','MAIL','GEO')

Elle n'en finit pas, j'ai pourtant rajouté les indexs sur PSU_CODE et FTP_CODE mais rien n'y fait.

Quelqu'un a t'il une solution pour un débutant qui galère ? :lol:

Merci d'avance.

21/03/2006, 11h23
SQLpro

Ajouter des des index à une table temporaire locale n'a pas d'intérêt...

Cela va ralentir notablement l'insertion.

Que voulez vous faire avec tout ceci ???

De plus vous faites un group by alors que je n'ai pas vu de fonction d'agrégation...

Faites vos jointure avec des JOIN non dans la clause WHERE !
http://sqlpro.developpez.com/cours/sqlaz/jointures/
21/03/2006, 11h36
mpeppler

Le DISTINCT combiné avec le GROUP BY est suspet - il est probable que vous genèrez un produit cartesien qui est caché par le DISTINCT.

Il faut bien s'assurer que la clause GROUP BY corresponde à tout les champs qui ne sont pas agrégé dans le SELECT, y compris les CASE...

Pour le reste - les indexes créés sur la table temporaires ne servent évidemment à rien dans le bout de code que vous montrez, mais je suppose qu'ils ont une fonction dans une autre partie de votre traitement.

Michael

Essayez :

Code:

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
29
30
31
32
33
34
35
36
37
38
39
40
41
create table #LISTE_CLIENTS
   (
      CLI_ID              numeric(16),
      CLI_ORIGINE         char(1),
      ANNEE               int,
      MOIS                int,
      PGE_Q_LBL_PRODUIT   varchar(32),
      PSU_CODE            char(4),
      FTP_CODE            char(4),
      FAC_CA              money,
      FAC_VOL             numeric(30,2)
   )
 
Insert into #LISTE_CLIENTS
 
select distinct T1.CLI_ID, T1.CLI_ORIGINE, T2.FAC_ANNEE, T2.FAC_MOIS,
      case when T3.PGE_Q_LBL_PRODUIT is NULL or T3.PGE_Q_LBL_PRODUIT = '' then "NON IDENTIFIE" else T3.PGE_Q_LBL_PRODUIT end,
      case when T3.PSU_CODE is NULL or T3.PSU_CODE='' then "N.I" else T3.PSU_CODE end,
      case when T3.FTP_CODE is NULL or T3.FTP_CODE='' then "N.I" else T3.FTP_CODE end,
      sum(T2.FAC_CA), sum(T2.FAC_VOL)
 
from  dm_Horus_exp.dbo.MS_CLIENT T1
         INNER JOIN dm_Horus_exp.dbo.MS_FACTURE T2
               ON T1.CLI_ID=T2.CLI_ID
         INNER JOIN dm_Horus_exp.dbo.MS_FAC_PRESTA T4
               ON T2.FAC_NO=T4.FAC_NO
         INNER JOIN dm_Horus_exp.dbo.MS_PRESTATION T3
               ON T4.PGE_ID=T3.PGE_ID
 
where (T3.PSU_CODE in ('THEM','GEO','CFCA','CCBK','CTIN','CSIM','CNET','CSOF','COPT','CCSC','AFFI','CPL','CPL2','FGPP') 
      or T3.FTP_CODE in ('ADR','MAIL','GEO'))   
      and datepart(yy, T2.FAC_DATE) = 2006
 
 
group by T1.CLI_ID, T1.CLI_ORIGINE, T2.FAC_ANNEE, T2.FAC_MOIS, T3.FTP_CODE, T3.PGE_Q_LBL_PRODUIT, T3.PSU_CODE
 
 
create index PK_MKT_LISTE_CLIENTS_id1 on #LISTE_CLIENTS (CLI_ID)
create index PK_MKT_LISTE_CLIENTS_id2 on #LISTE_CLIENTS (CLI_ORIGINE)
create index PK_MKT_EFFEC_PREC_id3 on #EFFEC_PREC (FTP_CODE)
create index PK_MKT_EFFEC_PREC_id4 on #EFFEC_PREC (PSU_CODE)

Votre OR ne me semble pas logique... Faîtes attention à l'ordre des prédicats dans les clauses logiques (filtre WHERE).

A +

21/03/2006, 12h03
Benjamin78

Pffiou quelle rapidité. Merci :D

Sinon pour le OR il peut ne pas sembler très logique, mais je n'ai pas le choix , la BDD sur laquelle je travaille est loin d'être au top.

Merci encorE.
21/03/2006, 12h25
Benjamin78

La proc passe maintenant en moins de quinze minutes.

C'est parfait.

Merci.