Evolution #1979
Remplacer "minidom" par "etree" pour la lecture des dictionnaires
Description
Il y a de fortes chances pour qu'on y gagne en performance...
Révisions associées
see #1979 Can hide ticket category to post-only users (to be improved)
see #1979 remove TicketCategory::dropdown and add a condition parameter to dropdownValue
- creole/lxml_parser.py : nouveau parseur
- creole/dom_parser.py : ancien parseur (déprécié)
- creole/cfgparser.py : modification des imports
Ménage dans cfgparser.py (ref #1979 @30m)
correction du parsing lxml (separateurs sans texte) + retour read_string pour Zéphir (ref #1979 @30m)
lxml_parser : correction du calcul de l'attribut mode si pas de valeur dans le xml (ref #1979)
lxml_parser : encore une erreur sur le calcul des separators (ref #1979)
Historique
#1 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Assigné à mis à Gwenael Remond
#2 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.3 - 02 RC à Mises à jour 2.2.3 - 03 RC
#3 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.2.3 - 03 RC à Mises à jour 2.3 - 02 RC
#4 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.3 - 02 RC à Mises à jour 2.3 - 02 Stable
#5 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.3 - 02 Stable à Mises à jour 2.3 - 03 RC
- Distribution mis à EOLE 2.3
#6 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.3 - 03 RC à Mises à jour 2.3.4 RC
- Temps estimé mis à 10.00 h
#7 Mis à jour par Joël Cuissinat il y a environ 12 ans
- Fichier lxml_parser.py Voir ajouté
En remplaçant le dom_parser.py par le fichier lxml_parser.py ci-joint, la différence n'est pas évidente :(
=> à tester sur un AmonEcole
#8 Mis à jour par Gwenael Remond il y a environ 12 ans
pour que ça gagne du temps c'est moins le fait d'utiliser lxml par rapport à xml.etree, c'est surtout qu'il y a plein de findall() qu'il faut remplacer par du XPath, et là ça devrait gagner du temps.
#9 Mis à jour par Joël Cuissinat il y a environ 12 ans
- % réalisé changé de 0 à 20
Je comprends.
Mais c'est déjà un exploit en soi d'avoir réussi à changer le code et que ça marche toujours !
Enfin, pas tout à fait car le test unitaire : test_redefine.py plante :(
#10 Mis à jour par Joël Cuissinat il y a environ 12 ans
- Fichier lxml_parser.py Voir ajouté
#11 Mis à jour par Joël Cuissinat il y a environ 12 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 20 à 100
Appliqué par commit aabb787d958ca5683af6806beba7c111630dfe31.
#12 Mis à jour par Fabrice Barconnière il y a environ 12 ans
- Statut changé de Résolu à Fermé