r13916 - spip/ecrire/public

Notifications Commit Core
1

Author: cedric@yterium.com
Date: 2009-04-23 20:42:27 +0200 (jeu, 23 avr 2009)
New Revision: 13916

Log:
optimisation :
- les boucles sont implementees par un calculer_select+sql_fetchall+while(array_shift())
- la pagination debut_xx=@yy peut parser les resultats pour trouver la page sans relancer la requete une deuxieme fois

Modified:
   spip/ecrire/public/compiler.php
   spip/ecrire/public/criteres.php
   spip/ecrire/public/quete.php

Details: http://trac.rezo.net/trac/spip/changeset/13916

2

Le 23 avr. 09 à 20:42, cedric@yterium.com a écrit :

Author: cedric@yterium.com
Date: 2009-04-23 20:42:27 +0200 (jeu, 23 avr 2009)
New Revision: 13916

Log:
optimisation :
- les boucles sont implementees par un calculer_select+sql_fetchall+while(array_shift())

Si la requête ramène beaucoup de Rows, tu vas saturer la mémoire PHP illico.
Ca me parait une fausse bonne idée.

Emmanuel

3

Le 23 avril 2009 22:57, Emmanuel Saint-James
<Emmanuel.Saint-James@lip6.fr>a écrit :

Le 23 avr. 09 à 20:42, cedric@yterium.com a écrit :

Author: cedric@yterium.com

Date: 2009-04-23 20:42:27 +0200 (jeu, 23 avr 2009)
New Revision: 13916

Log:
optimisation :
- les boucles sont implementees par un
calculer_select+sql_fetchall+while(array_shift())

Si la requête ramène beaucoup de Rows, tu vas saturer la mémoire PHP
illico.
Ca me parait une fausse bonne idée.

Ah, voilà.
Tu m'a devancé et je voulais te poser la question car j'avais le doute sur
ce point.
J'ai volontairement séparé ce commit du précédent sur la pagination car je
n'étais pas tout a fait certain de sa pertinence.
Mais effectivement, tu as raison, je vais le revert.
Et au passage je vais mieux nommer sql_fecthall en sql_fetch_all ce qui sera
plus coherent avec

Cédric