Discussion :

[Esil2008]

Bonjour,

Je voudrais savoir comment pourrais-je optimiser la requete ci dessous. Elle prend trop de temps, plus de 25s. Sachant que la table CHILDORDERSTATUS contient 20.000 lignes, CHILDORDERID 6000 lignes et INSTRUMENT 5 l.

Code :

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

1
2
3
4
5
6
7
8
 
select AVG(cast(cos.PRICE as DOUBLE)) 
from CHILDORDERSTATUS cos 
where cos.CHILDORDERID 
    IN (SELECT co.CHILDORDERID FROM CHILDORDER co  
                                          WHERE co.SIDE = 's' 
                                          and  co.INSTRUMENTID 
                                                IN (SELECT inst.INSTRUMENTID FROM INSTRUMENT inst WHERE inst.VENUE = 'X' ))

Les tables j'ai pas le droit de les changer. La BD est une BD HSQLBD chargé en mémoire installée sur un poste distant.

Voila je crois que j'ai tout dit.

Meric pour votre aide

[Waldar]

Est-ce que vos tables sont indexées ?

[Esil2008]

Je viens de trouver un article très intéressant d'un membre de developpez.com
http://sqlpro.developpez.com/cours/optimiser/#L8
je pense que ça va bq m'aider Merci sqlpro

[Esil2008]

oui elles sont indexées, j'ai appliquée quelques conseils de SQLpro, j'ai pu gagné un temps fou par rapport à mes 25s mnt c'est 2à3s, c'est bq mieux.

Je voudrais bien gagner encore plus si c'est possible.

Les tables sont indexées mais tous les champs figurant ds le where ne sont pas indexés. Malheureusement j'ai pas la main sur cette base, il faut passer par qlq un d'autres pour rajouter cela au cas ou c'est très important.

voila ma nouvelle requete

Code :

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

select AVG(cast(cos.PRICE as DOUBLE)) from CHILDORDERSTATUS cos where EXISTS (SELECT * FROM CHILDORDER co where  co.SIDE = 'b' and cos.CHILDORDERID = co.CHILDORDERID and EXISTS (SELECT * FROM INSTRUMENT inst WHERE inst.VENUE = 'TQ' and co.INSTRUMENTID = inst.INSTRUMENTID ))

par exemple ds cette requete, la table CHILDORDER à parmi ces indexes CHILDORDERID (la clé) mais pas SIDE, meme chose pour INSTRUMENT , elle a INSTRUMENTID (la clé) mais pas VENUE

Merci

[Oishiiii]

Est-ce que des jointures ne serait pas plus performantes ? A tester.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT AVG(cast(cos.PRICE AS DOUBLE)) 
FROM CHILDORDERSTATUS cos 
WHERE EXISTS (
	SELECT * 
	FROM CHILDORDER co 
		JOIN INSTTRUMENT inst
			USING(INSTRUMENTID)
	WHERE  co.SIDE = 'b' 
	AND inst.VENUE = 'TQ'
	AND cos.CHILDORDERID = co.CHILDORDERID
)

Ou encore :

Code :

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

1
2
3
4
5
6
7
8
SELECT AVG(cast(cos.PRICE AS DOUBLE)) 
FROM CHILDORDERSTATUS cos
	JOIN CHILDORDER co
		ON cos.CHILDORDERID = co.CHILDORDERID
		JOIN INSTTRUMENT inst
			ON co.INSTRUMENTID = inst.INSTRUMENTID
WHERE  co.SIDE = 'b' 
AND inst.VENUE = 'TQ'

Un index sur CHILDORDERSTATUS(CHILDORDERID, SIDE) serait certainement bénéfique. Pour la table INSTRUMENT, je ne pense pas, elle est tellement petite que ce n'est pas pertinent.

[Snipah]

Que donnent les jointures?

Code :

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

1
2
3
4
5
6
7
SELECT 
      AVG(cast(cos.PRICE AS DOUBLE)) 
FROM CHILDORDERSTATUS cos 
JOIN CHILDORDER co ON cos.CHILDORDERID = co.CHILDORDERID
JOIN INSTRUMENT inst ON co.INSTRUMENTID = inst.INSTRUMENTID
WHERE co.SIDE = 'b'
AND inst.VENUE = 'TQ'

[Esil2008]

Merci pour vos retours

j'essaye de tester avec les join mais j'ai une erreur que je n'arrive pas à trouver

Code :

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

1
2
3
4
5
6
7
SELECT 
      AVG(cast(cos.PRICE AS DOUBLE)) 
FROM CHILDORDERSTATUS cos 
JOIN CHILDORDER co ON cos.CHILDORDERID = co.CHILDORDERID
JOIN INSTRUMENT inst ON co.INSTRUMENTID = inst.INSTRUMENTID
WHERE co.SIDE = 'b'
AND inst.VENUE = 'TQ'

peut être c'est une question de parenthèse

Unexpected token: JOIN in statement

[CinePhil]

Le message d'erreur semble vouloir dire qu'il n'accepte pas le mot JOIN, ce qui est pour le moins étrange étant donné que je lis ici que HSQLDB :

supporte une grande partie des standards SQL-92, SQL-99, et SQL:2003.

Essaie de préciser INNER JOIN au cas où il serait sourcilleux sur la syntaxe.

[Snipah]

Je ne connais pas hsqlbd, peut être en ajoutant le mot-clé inner, généralement facultatif :

Code :

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

1
2
3
4
5
6
7
SELECT 
      AVG(cast(cos.PRICE AS DOUBLE)) 
FROM CHILDORDERSTATUS cos 
INNER JOIN CHILDORDER co ON cos.CHILDORDERID = co.CHILDORDERID
INNER JOIN INSTRUMENT inst ON co.INSTRUMENTID = inst.INSTRUMENTID
WHERE co.SIDE = 'b'
AND inst.VENUE = 'TQ'

Sinon, jointure à l'ancienne :

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT 
      AVG(cast(cos.PRICE AS DOUBLE)) 
FROM 
     CHILDORDERSTATUS cos, 
     CHILDORDER co,
     INSTRUMENT inst
WHERE cos.CHILDORDERID = co.CHILDORDERID
AND co.INSTRUMENTID = inst.INSTRUMENTID
AND co.SIDE = 'b'
AND inst.VENUE = 'TQ'

Mais c'est moche.

Edit: Encore grillé, c'est pas mon jour

[Esil2008]

C'est ce que je me suis dit moi aussi mais j'ai cherché ds la doc HSQLDB et il utilise effectivement le JOIN dans leur exemple.

J'ai essayé avec le INNER JOIN, ça marche mnt j'ai trouvé aussi une autre erreur de frappe (INSTTRUMENT)

mnt il exécute la requête en moins de 100ms c trop cool

Merci bq

[Esil2008]

Je vais essayer de demander à rajouter plus d'index ds mes tables, pour essayer de gagner encore en perf

merci pour votre