Discussion :

[molo1987]

Salut tlm,

c comme le titre l'a indique, g un probleme de lenteur en niveau de ma requete, ca dure des minute juska avoir la resultat sinon ca bug (je parle des milliers de lignes), est ce que il'y'a une solution ou est ce que je dois midifier kelke choses au moins ?

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
SELECT * 
FROM 
(
	SELECT p.id, p.domain,
		(
			SELECT SUM(s1.uniques) AS traffic 
			FROM dt_parked_stats s1 
			WHERE s1.uniques > 0 
				AND s1.owner = p.owner 
				AND s1.domain = p.domain 
				AND s1.date >= DATE_SUB(CURRENT_DATE(), INTERVAL 7 DAY)
		) AS traffic, 
		(
			SELECT c.country 
			FROM ".TABLE_PREFIX."cached_countries c 
			WHERE c.domain = p.domain 
				AND LOWER(c.country) != 'x1' 
			GROUP BY c.country 
			ORDER BY SUM(c.uniques) DESC 
			LIMIT 1
		) as maincountry 
	FROM dt_parked p 
	WHERE p.blocked = '3'
) tmp 
ORDER BY traffic DESC 
LIMIT 1500;

Merci d'avance

[BPiero]

Salut,

Tu devrai utiliser des LEFT JOIN.

@+

Piero

[ericd69]

salut,

il a raison, tes sous-requête comme celles-là qui dépend de la requête principale doivent être transformées en jointure avec pour condition ce que tu mets dans leur where... et en plaçant des index sur les colonnes de jointure

déjà on réécrit ta pauvre requête en l'indentant mieux:

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
SELECT * FROM
	(SELECT p.id, p.domain,
		(SELECT SUM(s1.uniques) AS traffic
			FROM dt_parked_stats s1
			WHERE s1.uniques>0 AND s1.owner=p.owner AND s1.domain=p.domain AND s1.date >= DATE_SUB(CURRENT_DATE(), INTERVAL 7 DAY)) AS traffic,
		(SELECT c.country
			FROM ".TABLE_PREFIX."cached_countries c
			WHERE c.domain=p.domain AND LOWER(c.country)!='x1'
			GROUP BY c.country
			ORDER BY SUM(c.uniques) DESC LIMIT 1) as maincountry
	FROM dt_parked p
	WHERE p.blocked='3'
) tmp
ORDER BY traffic DESC
LIMIT 1500;

on se rend ainsi compte que la requête la plus externe... ne sert pas à grand chose...

si tu utilise un nom de colonne identique à une mot clé, anti-quote le: `date`...

tu DOIS mettre un group by dans une requête pour préciser les groupement sur les colonnes où il n'y a pas de fonction de groupage...

cela devrait te donner:

Code sql :

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
SELECT p.id, p.domain, s1.traffic AS traffic, c.country as maincountry
FROM dt_parked p
inner join 
	(select sum(s1.uniques) as traffic, owner,domain
	where uniques>0 and date >= DATE_SUB(CURRENT_DATE(), INTERVAL 7 DAY)
	group by domain,owner) s1
	on s1.domain=p.domain and s1.owner=p.owner
inner join
	(SELECT country, domain, SUM(uniques) as somme
	FROM ".TABLE_PREFIX."cached_countries
	where LOWER(country)!='x1'
	group by country, domain
	order by somme desc) c
	on c.domain=p.domain
WHERE p.blocked='3'
ORDER BY traffic DESC
LIMIT 1500;

à voir l'ordre dans les group by et si tu veux des inner (entrées devant exister et coreespondre dans la table de droite ET gauche pour générer une sortie) ou des left join (il faut au moins une entrée dans la table de gauche pour générer une sortie)...

en gros tu transforme tes sous requête en une requête de jointure où tu appliques les conditions n'étant pas des conditions de jointure et dans le on celles qui le sont...

ne jamais utiliser * surtout avec des jointures...

[molo1987]

la requete est devenu avec left/inner join comme tu l a ecrit ici plus plus plus lourd qu'avant

[CinePhil]

Les tables sont-elles indexées ?

[molo1987]

Les tables sont-elles indexées ?

oui oui bien sure

[CinePhil]

Montre nous le plan d'exécution de la requête (résultat de EXPLAIN la_requete).

[redoran]

Salam , je crois qu'il faut optimisé tes requêtes :
séparé les sous requêtes qui donne un volume plus important de données et les traités a par pour voir le temps d’exécution ensuite les montées une a une en partant de l’intérieur vers l'extérieur.

[molo1987]

1 PRIMARY <derived2> system NULL NULL NULL NULL 0 const row not found
1 PRIMARY p ALL NULL NULL NULL NULL 617643 Using where
1 PRIMARY <derived3> ALL NULL NULL NULL NULL 1
4 DEPENDENT SUBQUERY dt_parked_keywords unique_subquery PRIMARY PRIMARY 4 func 1 Using index
3 DERIVED dt_cached_countries ALL NULL NULL NULL NULL 1155676 Using where; Using temporary; Using filesort
2 DERIVED s1 range date date 3 NULL 1 Using where; Using temporary; Using filesort

[CinePhil]

Je vois surtout des clés primaires mais pas beaucoup d'index !

Tu peux donner la définition complète des tables impliquées dans la requête ?

Dis-nous aussi ce que tu cherches à obtenir avec cette requête.

[redoran]

si ma lecture est exacte :

PRIMARY <derived2> system NULL NULL NULL NULL 0 const row not found

là il ne trouve pas les lignes a a examinées. !!!
je crois qu'il manque d'autres résultats dans le plan d'exécution non!!!

[molo1987]

pardon tlm, mais g pas une tres bonnes connaissances la , c un peu nouveau ces truc la pour moi

[ericd69]

comme le dit cinéphil tu n'as que les index de clés primaires or tu dois avoir des index sur les colonnes servant aux jointures donc ici au moins:
domain, owner

la lenteur vient dans tous les cas de la requête sur les pays...

j'avoue que tu as l'air d'avoir une usine à gaz à ce niveau

rien qu'au niveau de l'optimisation des where...
vu que:

Code sql :

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

DATE_SUB(CURRENT_DATE(), INTERVAL 7 DAY)

c'est aussi:

Code sql :

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

CURRENT_DATE() - INTERVAL 7 DAY

et que tu construis ta à la volée:

Code php :

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

".TABLE_PREFIX."

donc autan mettre directement la date au format "aaaa-mm-jj" dans le where

déjà essaye ça

et mets nous les create des tables

s'il n'utilise pas les index que tu auras rajouté... il y a une solution pour lui faire utiliser