Discussion :

[ajuveneton]

Bonjour,

Dans ma base access, j'ai une requête dont l'exécution est particulièrement longue:

Code :

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

1
2
3
4
5
SELECT DT.Numéro_dt, DT.Problème_signalé, DT.Nom_opérateur_prob, DT.Problème_signalé, DT.Date_prob, Organe.Organe, Machine.Machine, Section.Section, Organe.Numero_organe
FROM ([Section] INNER JOIN (Machine INNER JOIN Organe ON Machine.Machine = Organe.Machine) ON Section.Section = Machine.Section) INNER JOIN DT ON Organe.Numero_organe = DT.Num_org
GROUP BY DT.Numéro_dt, DT.Problème_signalé, DT.Nom_opérateur_prob, DT.Problème_signalé, DT.Date_prob, Organe.Organe, Machine.Machine, Section.Section, Organe.Numero_organe
HAVING (((DT.Numéro_dt) In (SELECT [Action].[Num_DT] FROM [Action] WHERE Action.Num_DT NOT IN (SELECT Action.Num_DT FROM [Action] WHERE [Action].[Etat]<>"Terminé" GROUP BY Action.Num_DT) GROUP BY [Action].[Num_DT])) AND ((Machine.Machine)=IIf([Tri_machine] Is Not Null,[Tri_machine],[Machine].[Machine])) AND ((Section.Section)=IIf([Tri_section] Is Not Null,[Tri_section],[Section].[Section])) AND ((Organe.Numero_organe)=IIf([Tri_organe] Is Not Null,[Tri_organe],[Organe].[Numero_organe])))
ORDER BY DT.Numéro_dt DESC;

Je cherche donc à l'optimiser. Or en y regardant de plus près, il me semble que la partie en gras est superflue. Si je la supprime, la requête devient:

Code :

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

1
2
3
4
5
SELECT DT.Numéro_dt, DT.Problème_signalé, DT.Nom_opérateur_prob, DT.Problème_signalé, DT.Date_prob, Organe.Organe, Machine.Machine, Section.Section, Organe.Numero_organe
FROM ([Section] INNER JOIN (Machine INNER JOIN Organe ON Machine.Machine = Organe.Machine) ON Section.Section = Machine.Section) INNER JOIN DT ON Organe.Numero_organe = DT.Num_org
GROUP BY DT.Numéro_dt, DT.Problème_signalé, DT.Nom_opérateur_prob, DT.Problème_signalé, DT.Date_prob, Organe.Organe, Machine.Machine, Section.Section, Organe.Numero_organe
HAVING (((DT.Numéro_dt) Not In (SELECT Action.Num_DT FROM [Action] WHERE [Action].[Etat]<>"Terminé" GROUP BY Action.Num_DT)) AND ((Machine.Machine)=IIf([Tri_machine] Is Not Null,[Tri_machine],[Machine].[Machine])) AND ((Section.Section)=IIf([Tri_section] Is Not Null,[Tri_section],[Section].[Section])) AND ((Organe.Numero_organe)=IIf([Tri_organe] Is Not Null,[Tri_organe],[Organe].[Numero_organe])))
ORDER BY DT.Numéro_dt DESC;

Pensez-vous que ce soit équivalent?

NOTE: Pour info ma base est une MAO (Maintenance Assistée par Ordinateur). Cette requête sert à trier les Demandes de Travail (DT) soldées c'est à dire dont toutes les actions ont été soldées (1 DT = plusieurs actions)

D'avance merci de votre aide

[SQLpro]

Commencez par supprimer toutes les clauses GROUP BY et leur contenu.
Placez les filtres dans les WHERE. Un GROUP BY n'a d'intérêt que si vous avez des calculs d'agrégats statistiques. Or comme il n'y en a pas cela fait perdre du temps à votre traitement en pure perte.

Postez à nouveau votre requête en l'identant de manière lisible pour que nous puissions nous pencher dessus à nouveau après cet élagage.

A +

[ajuveneton]

Voici ce que ça donne après avoir suivi vos conseils.

Pour l'indentation ce n'est pas forcément très orthodoxe mais ça me semble malgré tout plus lisible...

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
SELECT 	DT.Problème_signalé, DT.Nom_opérateur_prob, DT.Problème_signalé, DT.Date_prob, Organe.Organe, Machine.Machine
 
FROM 	(
		[Section]
		INNER JOIN 	(Machine
				INNER JOIN Organe
				ON Machine.Machine = Organe.Machine)
		ON Section.Section = Machine.Section
	)
	INNER JOIN DT
	ON Organe.Numero_organe = DT.Num_org
 
WHERE 	(
		((DT.Numéro_dt) NOT IN 	(SELECT Action.Num_DT
					FROM [Action]
					WHERE [Action].[Etat]<>"Terminé"
					GROUP BY Action.Num_DT)
	)
 	AND ((Machine.Machine)=IIf([Tri_machine] Is Not Null,[Tri_machine],[Machine].[Machine]))
	AND ((Section.Section)=IIf([Tri_section] Is Not Null,[Tri_section],[Section].[Section]))
	AND ((Organe.Numero_organe)=IIf([Tri_organe] Is Not Null,[Tri_organe],[Organe].[Numero_organe])))
 
ORDER BY DT.Numéro_dt DESC;

[Magnus]

Pourquoi mettre certains noms d'objets entre crochets surtout si ce n'est pas systématique ?
De plus, bon nombre de vos parenthèses sont inutiles.
Supprimez-les même si ça n'augmentera pas les performances, votre requête n'en sera que plus lisible.
Voici votre requête épurée :

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
SELECT   DT.PROBLèME_SIGNALé,
         DT.NOM_OPéRATEUR_PROB,
         DT.PROBLèME_SIGNALé,
         DT.DATE_PROB,
         ORGANE.ORGANE,
         MACHINE.MACHINE
FROM     SECTION
         INNER JOIN MACHINE
                    INNER JOIN ORGANE
                      ON MACHINE.MACHINE = ORGANE.MACHINE
           ON SECTION.SECTION = MACHINE.SECTION
         INNER JOIN DT
           ON ORGANE.NUMERO_ORGANE = DT.NUM_ORG
WHERE    DT.NUMéRO_DT NOT IN (SELECT   ACTION.NUM_DT
                              FROM     ACTION
                              WHERE    ACTION.ETAT <> "Terminé"
                              GROUP BY ACTION.NUM_DT)
         AND MACHINE.MACHINE = IIF(TRI_MACHINE IS NOT NULL,TRI_MACHINE,MACHINE.MACHINE)
         AND SECTION.SECTION = IIF(TRI_SECTION IS NOT NULL,TRI_SECTION,SECTION.SECTION)
         AND ORGANE.NUMERO_ORGANE = IIF(TRI_ORGANE IS NOT NULL,TRI_ORGANE,ORGANE.NUMERO_ORGANE)
ORDER BY DT.NUMéRO_DT DESC

Enfin concernant votre question, SQLpro est BIEN plus calé que moi pour vous donner des conseils, néanmoins en analysant le plan de votre requête, voici les points sur lesquels vous pourriez agir :
- remplacer les [NOT] IN par [NOT] EXISTS
- remplacer les IIF par des CASE WHEN

Remarque : je ne suis pas catégorique sur le fait que la suppression des parenthèses que vous avez mis dans la clause FROM ne modifie pas le jeu de résultats obtenu en sorti.
Cela m'étonnerait mais faîtes-le vous confirmer par un pro